Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gjl-boxop-draw
DEFAULT BRANCH: main
Repo Added
Files 872
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-14840-2bf1b824139b856514be8cac1d83a55453924eb0
branch: gh-readonly-queue/main/pr-14840-2bf1b824139b856514be8cac1d83a55453924eb0
CHANGE BRANCH
x

coverage: 88.388% (-0.02%) from 88.405%
17243441617

push

github

web-flow 
Burn inverse cancellation (#14840)

* Add Python free path to inverse_cancellation

This commit adds a Python free path to the inverse cancellation
transpiler pass. Previously, the only way to run the pass required Python
because the pass handles arbitrary gates to cancel which can include
custom gates from Python. This meant to validate the gates were equal we
would need the Python interpreter. This commit opts to add a new
entrypoint for Rust as working under the constraints that we're only
having StandardGates can lead to a more efficient implementation.
Additionally, for the preset pass managers we always used the same fixed
set of standard gates which don't take parameters to cancel. Since the
initial goal of the C API is to expose the preset pass managers this new
entrypoint only uses this set which simplifies the checking necessary.
This provides an optimized path for the preset pass managers too. For the
standalone C function in #14444 we don't need to make the pass
configurable. If we decide eventually to expand this it is trivial to add
an updated entrypoint, but for the first MVP it's not necessary.

Since this new entrypoint is also more efficient for the use case of the
preset pass managers the Python side pass is updated to use it in the
default case. This enables the preset pass managers to run the pass
faster.

Related to #14444

* Fix copy paste error in inverse pair path

The inverse pair path was incorrectly checking the same gate twice
instead of the gate and the next one in a run. This was just a copy and
paste error where code was supposed to operate on n and n+1 but it was
doing n twice. This commit fixes this and expands the testing to ensure
we have direct coverage of the new code path instead of relying on the
transpiler tests testing indirectly.

* Apply suggestions from code review

Co-authored-by: Alexander Ivrii <alexi@il.ibm.com>

* Rename collect_1q_runs_by() -> collect_runs_by()

* Fix short circuit conditions for inve... (continued)

11 of 13 new or added lines in 1 file covered. (84.62%)

29 existing lines in 3 files now uncovered.

90118 of 101957 relevant lines covered (88.39%)

489496.89 hits per line

Relevant lines Covered
101957 RELEVANT LINES 90118 COVERED LINES
489496.89 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14840-2bf1b824139b856514be8cac1d83a55453924eb0

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
17243441617 gh-readonly-queue/main/pr-14840-2bf1b824139b856514be8cac1d83a55453924eb0 Burn inverse cancellation (#14840) * Add Python free path to inverse_cancellation This commit adds a Python free path to the inverse cancellation transpiler pass. Previously, the only way to run the pass required Python because the pass handles ... push web-flow github
88.39
See All Builds (14193)