Qiskit / qiskit / 11052352075

88%

Build Type

push

github

Committed by web-flow

Commit Message

Fully port BarrierBeforeFinalMeasurements to rust (#13220)

* Fully port BarrierBeforeFinalMeasurements to rust

This commit migrates the BarrierBeforeFinalMeasurements transpiler pass
to operate fully in Rust. The full path of the transpiler pass now never
leaves Rust until it has finished modifying the DAGCircuit. The one
exception is when the `label` is not set then we still call
`MergeAdjacentBarriers` in the python code of the pass.

This is the first step in the improvement of the performance of this
pass. We can easily leverage multhreading to potentially parallelize the
analysis portion of this pass that searches for the final operations and
returns the set of indices. But this is blocked on #13219 which prevents
us from accessing the PackedInstructions stored in the DAGCircuit in a
multithreaded context.

This commit also fixes an issue related to shared references in
the disjoint_utils module around barrier labels. The combine_barriers()
function was incorrectly mutating the label by reference which wouldn't
persist in the DAG, and this was causing failures after the barrier was
originally generated in Rust with this pass now.

Fixes #12253

* Remove unused imports

Run Details

90 of 93 new or added lines in 6 files covered. (96.77%)

33 existing lines in 3 files now uncovered.

73902 of 83236 relevant lines covered (88.79%)

359939.16 hits per line