Qiskit / qiskit
90%
main: 89%

LAST BUILD BRANCH: remove_deprecated_function_arguments/1
DEFAULT BRANCH: main
Repo Added
Files 914
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 mergify/bp/stable/1.2/pr-12727
branch: mergify/bp/stable/1.2/pr-12727
CHANGE BRANCH
x

coverage: 89.953% (+0.008%) from 89.945%
10200362536

Pull #12881

github

mergify[bot] 
'Peephole' optimization - or: collecting and optimizing two-qubit blocks - before routing (#12727)

* init

* up

* up

* Update builtin_plugins.py

* Update builtin_plugins.py

* reno

* Update builtin_plugins.py

* Update builtin_plugins.py

* Update peephole-before-routing-c3d184b740bb7a8b.yaml

* neko check

* check neko

* Update builtin_plugins.py

* test neko

* Update builtin_plugins.py

* Update builtin_plugins.py

* Update builtin_plugins.py

* lint

* tests and format

* remove FakeTorino test

* Update peephole-before-routing-c3d184b740bb7a8b.yaml

* Apply suggestions from code review

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* comments from code review

* fix precision

* up

* up

* update

* up

* .

* cyclic import

* cycl import

* cyl import

* .

* circular import

* .

* lint

* Include new pass in docs

* Fix Split2QUnitaries dag manipulation

This commit fixes the dag handling to do the 1q unitary insertion.
Previously the dag manipulation was being done manually using the
insert_node_on_in_edges() rustworkx method. However as the original node
had 2 incoming edges for each qubit this caused the dag after running
the pass to become corrupted. Each of the new 1q unitary nodes would end
up with 2 incident edges and they would be in a sequence. This would result
in later passes not being able to correctly understand the state of the
circuit correctly. This was causing the unit tests to fail. This commit
fixes this by just using `substitute_node_with_dag()` to handle the
node substition, while doing it manually to avoid the overhead of
checking is probably possible, the case where a unitary is the product
of two 1q gates is not very common so optimizing it isn't super
critical.

* Update releasenotes/notes/peephole-before-routing-c3d184b740bb7a8b.yaml

* stricter check for doing split2q

* Update qiskit/transpiler/preset_passmanagers/builtin_plugins.py

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* code r... (continued)
Pull Request #12881: 'Peephole' optimization - or: collecting and optimizing two-qubit blocks - before routing (backport #12727)

59 of 62 new or added lines in 4 files covered. (95.16%)

14 existing lines in 2 files now uncovered.

66738 of 74192 relevant lines covered (89.95%)

234796.4 hits per line

Relevant lines Covered
74192 RELEVANT LINES 66738 COVERED LINES
234796.4 HITS PER LINE
Source Files on mergify/bp/stable/1.2/pr-12727

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10200362536 mergify/bp/stable/1.2/pr-12727 'Peephole' optimization - or: collecting and optimizing two-qubit blocks - before routing (#12727) * init * up * up * Update builtin_plugins.py * Update builtin_plugins.py * reno * Update builtin_plugins.py * Update builtin_plugins.py * U... Pull #12881 mergify[bot] github
89.95
See All Builds (8713)