Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15122-bfd268c1b109b7023bf0b294f289c96d55000a00
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-14668-6b4477fe2e42047aba7babe4c834f0b22a946737
branch: gh-readonly-queue/main/pr-14668-6b4477fe2e42047aba7babe4c834f0b22a946737
CHANGE BRANCH
x

coverage: 87.755% (-0.007%) from 87.762%
16277835384

push

github

web-flow 
Add VF2Layout standalone function to the C API (#14668)

* Add VF2Layout standalone function to the C API

This commit is the first transpiler pass that is part of the C API. VF2
layout is already rust native and it doesn't require Python in the
execution of the pass. As the first transpiler pass in the C API this
introduces some new infrastructure to support this. The goal of the
standalone function is not so much for pass manager composibility
but to have the one off execution of passes This is the difference
between in python doing:

VF2Layout(...)(qc)

and

VF2Layout(...).run(dag)

but the medium term goal is to use the standalone functions as
validation that we can run a pass from C without a runtime Python
dependency. The eventual goal is to expose a full path "compile()"
or "transpile()" function.

This starts the naming convention for standalone passes that process a
CircuitData. These functions will be of the form:

qk_transpiler_pass_standalone_pass_name

This will be differentiated from pass functions that work with a
DAGCircuit, when we have the DAGCircuit exposed to C. Those will be
of the form:

qk_transpiler_pass_name (e.g. qk_transpiler_vf2_layout)

or

qk_transpiler_pass_pass_name (e.g. qk_transpiler_pass_vf2_layout)

we can decide on the specific dag naming convention when we start to
work on that.

To faciliate running the pass without Python there is small tweak needed
to the CircuitData -> DAGCircuit conversion path. The transpiler passes
only has a borrowed CircuitData since it is provided as a pointer from C
and the interface for the conversion needed to be inverted so that it
works with references instead of owned objects. Using owned objects is
fine from Python but isn't viable for Rust callers.

* Fix typo in docs config

* Use Rust native bit objects for circuit_to_dag instead of PyObject

* Apply suggestions from code review

Co-authored-by: Kevin Hartman <kevin@hart.mn>

* Update docs/cdoc/qk-transpiler-passes.rst

Co-auth... (continued)

91 of 106 new or added lines in 5 files covered. (85.85%)

17 existing lines in 4 files now uncovered.

81540 of 92918 relevant lines covered (87.75%)

548102.88 hits per line

Relevant lines Covered
92918 RELEVANT LINES 81540 COVERED LINES
548102.88 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14668-6b4477fe2e42047aba7babe4c834f0b22a946737

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
16277835384 gh-readonly-queue/main/pr-14668-6b4477fe2e42047aba7babe4c834f0b22a946737 Add VF2Layout standalone function to the C API (#14668) * Add VF2Layout standalone function to the C API This commit is the first transpiler pass that is part of the C API. VF2 layout is already rust native and it doesn't require Python in the e... push web-flow github
87.75
See All Builds (14220)