Qiskit
/
qiskit
/
11938613551
/
1
89%
main: 88%
main: 88%
|
Ran
|
Files
914
|
Run time
43s
|
Badge
Embed ▾
README BADGES
|
coverage: 88.917% (-0.01%) from 88.931%
push
github
web-flowFix basis_count checking in ConsolidateBlocks (#13463) (#13465) The ConsolidateBlocks pass was ported to rust in #13368 and as part of that implementation a small behavior difference between the rust and python interfaces was causing the pass to not work correctly with non-CX gates. The internal 2q decomposer interface stores a sentinel string for the kak gate which is used to tell the python space constructor use the python defined gate object. However in the pass code we weren't factoring this difference in, and for non-CX gates we were evaluating the basis count as the number of gates with that sentinel value name (which is almost always zero) and this was preventing the pass from consolidating many blocks that should have been. This commit fixes this issue by taking the name from python space and passing it through to the rust portion of the code and using that for the comparison. Fixes #13459 (cherry picked from commit c7924262f) Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
79056 of 88910 relevant lines covered (88.92%)
355949.63 hits per line
