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-12995-576efcf14a4be392cbcdb3c75d2838a440f73b13
branch: gh-readonly-queue/main/pr-12995-576efcf14a4be392cbcdb3c75d2838a440f73b13
CHANGE BRANCH
x

coverage: 89.173% (+0.02%) from 89.153%
10707349892

push

github

web-flow 
Oxidize Commutation Analysis (#12995)

* init

* up

* lint

* .

* up

* before cache

* with cache

* correct

* cleaned up

* lint reno

* Update Cargo.lock

* .

* up

* .

* revert op

* .

* .

* .

* .

* Delete Cargo.lock

* .

* corrected string comparison

* removed Operator class from operation.rs

* .

* Apply suggestions from code review

Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com>

* comments from code review

* Port DAGCircuit to Rust

This commit migrates the entirety of the `DAGCircuit` class to Rust. It
fully replaces the Python version of the class. The primary advantage
of this migration is moving from a Python space rustworkx directed graph
representation to a Rust space petgraph (the upstream library for
rustworkx) directed graph. Moving the graph data structure to rust
enables us to directly interact with the DAG directly from transpiler
passes in Rust in the future. This will enable a significant speed-up in
those transpiler passes. Additionally, this should also improve the
memory footprint as the DAGCircuit no longer stores `DAGNode`
instances, and instead stores a lighter enum NodeType, which simply
contains a `PackedInstruction` or the wire objects directly.

Internally, the new Rust-based `DAGCircuit` uses a `petgraph::StableGraph`
with node weights of type `NodeType` and edge weights of type `Wire`. The
NodeType enum contains variants for `QubitIn`, `QubitOut`, `ClbitIn`,
`ClbitOut`, and `Operation`, which should save us from all of the
`isinstance` checking previously needed when working with `DAGNode` Python
instances. The `Wire` enum contains variants `Qubit`, `Clbit`, and `Var`.

As the full Qiskit data model is not rust-native at this point while
all the class code in the `DAGCircuit` exists in Rust now, there are
still sections that rely on Python or actively run Python code via Rust
to function. These typically involve anything that uses `condition`,
control flow, classical vars, ca... (continued)

107 of 110 new or added lines in 6 files covered. (97.27%)

21 existing lines in 6 files now uncovered.

72599 of 81414 relevant lines covered (89.17%)

387637.33 hits per line

Relevant lines Covered
81414 RELEVANT LINES 72599 COVERED LINES
387637.33 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12995-576efcf14a4be392cbcdb3c75d2838a440f73b13

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10707349892 gh-readonly-queue/main/pr-12995-576efcf14a4be392cbcdb3c75d2838a440f73b13 Oxidize Commutation Analysis (#12995) * init * up * lint * . * up * before cache * with cache * correct * cleaned up * lint reno * Update Cargo.lock * . * up * . * revert op * . * . * . * . * Delete Cargo.lock * . * corrected... push web-flow github
89.17
See All Builds (9023)