Qiskit / qiskit-ibm-runtime / 26871469668

86%

push

github

Move Embeddings and CircuitSchedule to visualization module (#2864)

<!--
⚠️ The pull request title should be short detailed and understandable
for all.
⚠️ Also, please add a release note file using reno if the change needs
to be
  documented in the release notes.
⚠️ If your pull request fixes an open issue, please link to the issue.

✅ I have added the tests to cover my changes.
✅ I have updated the documentation accordingly.
✅ I have read the CONTRIBUTING document.
-->

### Summary

`Embeddings` and `CircuitSchedule` are only used by visualization
functions (in the case of `Embeddings`, it is also imported by
`NoiseLearner` - but only for visualization purposes). This PR moves
them to the `visualization` module, for better modularity, and
explicitly outside of `utils` to continue avoiding disparate on it.

### Details and comments

Related to #2749 and #2563 - there was actually an import cycle between
`draw_circuit_schedule_timings` and `CircuitSchedule` (as it traversed
`utils` that would surface if tackling #2563).

3 of 3 new or added lines in 3 files covered. (100.0%)

7855 of 9122 relevant lines covered (86.11%)

9.41 hits per line