Qiskit / qiskit
88%
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-14803-11625c9d2607a1f56a1ab62ba753acd285c4b90e
branch: gh-readonly-queue/main/pr-14803-11625c9d2607a1f56a1ab62ba753acd285c4b90e
CHANGE BRANCH
x

coverage: 87.665%. Remained the same
16834979106

push

github

web-flow 
Add Python free path to inverse_cancellation (#14803)

* 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 ... (continued)

125 of 132 new or added lines in 3 files covered. (94.7%)

19 existing lines in 3 files now uncovered.

82309 of 93890 relevant lines covered (87.67%)

546221.33 hits per line

Relevant lines Covered
93890 RELEVANT LINES 82309 COVERED LINES
546221.33 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14803-11625c9d2607a1f56a1ab62ba753acd285c4b90e

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
16834979106 gh-readonly-queue/main/pr-14803-11625c9d2607a1f56a1ab62ba753acd285c4b90e Add Python free path to inverse_cancellation (#14803) * 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 becaus... push web-flow github
87.67
See All Builds (13861)