Qiskit / qiskit
89%
main: 89%

LAST BUILD BRANCH: dag-concatenate
DEFAULT BRANCH: main
Repo Added
Files 896
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-13237-09d10065b90abccbd1ae446959aeaf07d9021818
branch: gh-readonly-queue/main/pr-13237-09d10065b90abccbd1ae446959aeaf07d9021818
CHANGE BRANCH
x

coverage: 88.723% (-0.03%) from 88.756%
11616480184

push

github

web-flow 
[Oxidize BasisTranslator]: Move the rest of the `BasisTranslator` to Rust.  (#13237)

* Initial: Move the rest of the `BasisTranslator` to Rust.

Fixes Port `BasisTranslator` to Rust #12246

This is the final act of the efforts to move the `BasisTranslator` transpiler pass into Rust. With many of the parts of this pass already living in Rust, the following commits attempt to bring in the final changes to allow complete operation of this pass in the Rust space (with of course some interaction with Python.)

Methodology:
The way this works is by keeping the original `BasisTranslator` python class, and have it store the rust-space counterpart (now called `CoreBasisTranslator`) which will perform all of the operations leveraging the existent Rust API's available for the `Target` (#12292), `EquivalenceLibrary`(#12585) and the `BasisTranslator` methods `basis_search` (#12811) and `compose_transforms`(#13137).

All of the inner methods will have private visibility and will not be accessible to `Python` as they're intended to be internal by design.

By removing the extra layers of conversion we should be seeing a considerable speed-up, alongside all of the other incremental improvements we have made.

Changes:

- Add the pyo3 class/struct `BasisTranslator` that will contain allof the main data used by the transpiler pass to perform its operation.
- Convert the `target_basis` into a set manually from python before sending it into the Rust space.
- Remove the exposure of `basis_search` and `compose_transforms` to python.
- Change `basis_search` so that it accepts references to `HashSet` instances instead of accepting a `HashSet<&str>` instance.
- Change inner method's visibility for `basis_search` and `compose_transform` modules in rust.
- Expose the exception imports from `Target` to the `accelerate` crate.
- Expose `DAGCircuit::copy_empty_like` to the rest of the crates.
- Remove all of the unused imports in the Python-side `BasisTranslator`.

Blockers:
- [ ]... (continued)

570 of 651 new or added lines in 2 files covered. (87.56%)

25 existing lines in 7 files now uncovered.

76374 of 86081 relevant lines covered (88.72%)

359395.21 hits per line

Relevant lines Covered
86081 RELEVANT LINES 76374 COVERED LINES
359395.21 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13237-09d10065b90abccbd1ae446959aeaf07d9021818

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11616480184 gh-readonly-queue/main/pr-13237-09d10065b90abccbd1ae446959aeaf07d9021818 [Oxidize BasisTranslator]: Move the rest of the `BasisTranslator` to Rust. (#13237) * Initial: Move the rest of the `BasisTranslator` to Rust. Fixes Port `BasisTranslator` to Rust #12246 This is the final act of the efforts to move the `BasisT... push web-flow github
88.72
See All Builds (8468)