Qiskit
/
qiskit
/
18398415308
/
1
88%
main: 88%
main: 88%
|
Ran
|
Files
867
|
Run time
28s
|
Badge
Embed ▾
README BADGES
|
coverage: 88.399% (+0.001%) from 88.398%
push
github
web-flowFix incorrect mapping of ControlFlow block qargs in `BasisTranslator` (#15143) (#15150) * Fix: `ControlFlowOP`/`BasisTranslator` mismatch - When extracting the basis of the target and applying the translations onto the dag, the `BasisTranslator` would not use the absolute indices of the operating qubits when a control flow operation block was processed during recursion. The following commit adds a mapping to the following `BasisTranslator` methods: - `extract_target_basis_circuit` which only gets called during a recursion step. - `apply_translation` in which it is always `None` unless a recursion step happens. The provided mapping will correctly direct local block qubit indices into the absolute ones stored by the original circuit from which the operation comes from. This should help fix #13728. * Fix: Add check for unique qargs. - Map the unique qargs during `apply_translation` in `BasisTranslator`. * Test: Add simplified reproducer. * Revert: Error message in `apply_translation` * Fix: Add release note (cherry picked from commit 21a8dec17) Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com>
92338 of 104456 relevant lines covered (88.4%)
545290.79 hits per line
