Qiskit / qiskit
89%
main: 89%

LAST BUILD BRANCH: BD-add-alt-text
DEFAULT BRANCH: main
Repo Added
Files 913
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-12959-31fbcacccabf464a56652edb082cb7e69df71622
branch: gh-readonly-queue/main/pr-12959-31fbcacccabf464a56652edb082cb7e69df71622
CHANGE BRANCH
x

coverage: 89.152% (-0.03%) from 89.185%
10635763010

push

github

web-flow 
Fully port Optimize1qGatesDecomposition to Rust (#12959)

* Fully port Optimize1qGatesDecomposition to Rust

This commit builds off of #12550 and the other data model in Rust
infrastructure and migrates the Optimize1qGatesDecomposition pass to
operate fully in Rust. The full path of the transpiler pass now never
leaves rust until it has finished modifying the DAGCircuit. There is
still some python interaction necessary to handle parts of the data
model that are still in Python, mainly calibrations and parameter
expressions (for global phase). But otherwise the entirety of the pass
operates in rust now.

This is just a first pass at the migration here, it moves the pass to be
a single for loop in rust. The next steps here are to look at operating
the pass in parallel. There is no data dependency between the
optimizations being done by the pass so we should be able to the
throughput of the pass by leveraging multithreading to handle each run
in parallel. This commit does not attempt this though, because of the
Python dependency and also the data structures around gates and the
dag aren't really setup for multithreading yet and there likely will
need to be some work to support that (this pass is a good candidate to
work through the bugs on that).

Part of #12208

* Tweak control_flow_op_nodes() method to avoid dag traversal when not necessary

* Store target basis set without heap allocation

Since we only are storing 12 enum fields (which are a single byte) using
any heap allocated collection is completely overkill and will have more
overhead that storing a statically sized array for all 12 variants. This
commit adds a new struct that wraps a `[bool; 12]` to track which
basis are supported and an API for tracking this. This simplifies the
tracking of which qubit supports which EulerBasis, it also means other
internal users of the 1q decomposition have a simplified API for working
with the euler basis.

* Remove From trait for Qubit->PhysicalQubit conver... (continued)

306 of 351 new or added lines in 6 files covered. (87.18%)

26 existing lines in 5 files now uncovered.

71825 of 80565 relevant lines covered (89.15%)

399328.66 hits per line

Relevant lines Covered
80565 RELEVANT LINES 71825 COVERED LINES
399328.66 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12959-31fbcacccabf464a56652edb082cb7e69df71622

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10635763010 gh-readonly-queue/main/pr-12959-31fbcacccabf464a56652edb082cb7e69df71622 Fully port Optimize1qGatesDecomposition to Rust (#12959) * Fully port Optimize1qGatesDecomposition to Rust This commit builds off of #12550 and the other data model in Rust infrastructure and migrates the Optimize1qGatesDecomposition pass to ope... push web-flow github
89.15
See All Builds (9023)