Qiskit / qiskit
89%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15094-96f5c08b3f54acfcc9552a34bf1bcc5402e07e7f
DEFAULT BRANCH: main
Repo Added
Files 868
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/main/pr-13368-6ed774395468b8992c2ea06d9b26f275b9632c11
branch: gh-readonly-queue/main/pr-13368-6ed774395468b8992c2ea06d9b26f275b9632c11
CHANGE BRANCH
x

coverage: 88.813% (+0.05%) from 88.763%
11708837361

push

github

web-flow 
Oxidize the ConsolidateBlocks pass (#13368)

* Oxidize the ConsolidateBlocks pass

This commit ports the consolidate blocks pass to rust. The logic remains
the same and this is just a straight porting. One optimization is that
to remove the amount of python processing the Collect2qBlocks pass is no
longer run as part of the preset pass managers and this is called
directly in rust. This speeds up the pass because it avoids 3 crossing
of the language boundary and also the intermediate creation of DAGNode
python objects. The pass still supports running with Collect2qBlocks for
backwards compatibility and will skip running the pass equivalent
internally the field is present in the property set.

There are potential improvements that can be investigated here such as
avoiding in place dag contraction and moving to rebuilding the dag
iteratively. Also changing the logic around estimated error (see #11659)
to be more robust. But these can be left for follow up PRs as they
change the logic.

Realistically we should look at combining ConsolidateBlocks for it's
current two usages with Split2qUnitaries and UnitarySynthesis into
those passes for more efficiency. We can improve the performance and
logic as part of that refactor. See #12007 for more details on this
for UnitarySynthesis.

Closes #12250

* Update test to count consolidate_blocks instead of collect_2q_blocks

* Fix lint

* Fix solovay kitaev test

* Add release note

* Restore 2q block collection for synthesis translation plugin

* Add rust native substitute method

* Fix final test failures

* Remove release note and test change

The test failure fixed by a test change was incorrect and masked a logic
bug that was fixed in a subsequent commit. This commit reverts that
change to the test and removes the release note attempting to document a
fix for a bug that only existed during development of this PR.

* Fix comment leftover from rust-analyzer

* Remove unused code

* Simplify control flow handling

*... (continued)

535 of 571 new or added lines in 9 files covered. (93.7%)

4 existing lines in 4 files now uncovered.

77335 of 87076 relevant lines covered (88.81%)

358741.84 hits per line

Relevant lines Covered
87076 RELEVANT LINES 77335 COVERED LINES
358741.84 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13368-6ed774395468b8992c2ea06d9b26f275b9632c11

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11708837361 gh-readonly-queue/main/pr-13368-6ed774395468b8992c2ea06d9b26f275b9632c11 Oxidize the ConsolidateBlocks pass (#13368) * Oxidize the ConsolidateBlocks pass This commit ports the consolidate blocks pass to rust. The logic remains the same and this is just a straight porting. One optimization is that to remove the amount... push web-flow github
88.81
See All Builds (13861)