Skip to content

* Aligned the CalibrationCreator to the removal of Qubit.index.#6139

Merged
mergify[bot] merged 6 commits into
Qiskit:masterfrom
eggerdj:calibration-builder-index
Apr 10, 2021
Merged

* Aligned the CalibrationCreator to the removal of Qubit.index.#6139
mergify[bot] merged 6 commits into
Qiskit:masterfrom
eggerdj:calibration-builder-index

Conversation

@eggerdj
Copy link
Copy Markdown
Contributor

@eggerdj eggerdj commented Apr 2, 2021

Summary

PR #6069 deprecated Bit.index which the CalibrationCreator was using. This PR aligns the CalibrationCreator to the new methodology.

Details and comments

The code now uses the methodology based on:

bit_indices = {bit: index for index, bit in enumerate(dag.qubits)}

@eggerdj eggerdj requested a review from a team as a code owner April 2, 2021 06:50
@eggerdj eggerdj requested a review from kdk April 2, 2021 06:51
@mtreinish mtreinish added the stable backport potential Make Mergify open a backport PR to the most recent stable branch on merge. label Apr 2, 2021
@kdk kdk self-assigned this Apr 6, 2021
@kdk kdk added the automerge label Apr 9, 2021
@mergify mergify Bot merged commit d22c506 into Qiskit:master Apr 10, 2021
mergify Bot added a commit that referenced this pull request Apr 10, 2021
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Co-authored-by: Kevin Krsulich <kevin.krsulich@ibm.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit d22c506)
mergify Bot added a commit that referenced this pull request Apr 10, 2021
…6139) (#6196)

* * Aligned the CalibrationCreator to the removal of Qubit.index. (#6139)

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Co-authored-by: Kevin Krsulich <kevin.krsulich@ibm.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit d22c506)

* Remove unnecessary release note

Co-authored-by: Daniel Egger <38065505+eggerdj@users.noreply.github.com>
Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@1ucian0 1ucian0 added the Changelog: None Do not include in the GitHub Release changelog. label Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. stable backport potential Make Mergify open a backport PR to the most recent stable branch on merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants