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-13335-f71d2011de04accaba65b3ee1a1b21705564687a
branch: gh-readonly-queue/main/pr-13335-f71d2011de04accaba65b3ee1a1b21705564687a
CHANGE BRANCH
x

coverage: 88.272% (+0.06%) from 88.213%
14525077100

push

github

web-flow 
Add new API for inserting multiple operations into the `DAGCircuit` in Rust. (#13335)

* Initial: Add `DAGCircuitConcat` struct and new insertion API.
- Add struct that allows continuous addition of instructions into the back of the `DAGCircuit` by performing edge modifications in the output nodes at the end of the additions.
- The new struct includes the following new methods:
    - `push_operation_back` which pushes a valid `PackedInstruction` into the back of the `DAGCircuit`. It behaves very similarly to `push_back`.
    - `apply_operation_back` which inserts a new operation into the back of the `DAGCircuit`. It has two variants which depend on the ownership of the bit indices (`Qubit`, `Clbit`).
    - `pack_instruction`: private method that basically does just that, it creates a valid instance of `PackedInstruction` based on other provided atributes.
- Modified `DAGCircuit::extend` to use this new API, without any considerable losses.

* Add: `OwnedOrSlice<T>`
- Add struct that allows usage of both slice or owned collections of bit indices.
- Leverage use of `as_concat()` in `circuit_to_dag`.

* Add: Optional qargs/cargs in `apply_operation_back`
- Leverage usage of new methods in `UnitarySynthesis` after #13141 merged.

* Fix: Leverage usage in `BasisTranslator`

* Fix: Replace `OnceCell` with `OnceLock`

* Fix: Remove `SliceOrOwned` in favor of `Cow`

* Fix: Expose the interners via references within `DAGCircuitConcat`.

* Refactor: `DAGCircuit::as_concat` to `DAGCircuit::into_concat`.
- Allow `DAGCircuitConcat` to take ownership of the original `DAGCircuit` it is being called from, and allow `DAGCircuitConcat::end` to return the original `DAGCircuit`.
- Bypass taking ownership of the `DAGCircuit` during `DAGCircuit::extend` calls by swapping with a temporary replacement.

* Fix: Remove inefficient code from `UnitarySynthesis`
- A previous commit added some inefficient code into the `apply_synth_dag` function in `UnitarySynthesis`. The newer co... (continued)

281 of 283 new or added lines in 3 files covered. (99.29%)

3 existing lines in 2 files now uncovered.

73824 of 83632 relevant lines covered (88.27%)

439197.32 hits per line

Relevant lines Covered
83632 RELEVANT LINES 73824 COVERED LINES
439197.32 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13335-f71d2011de04accaba65b3ee1a1b21705564687a

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
14525077100 gh-readonly-queue/main/pr-13335-f71d2011de04accaba65b3ee1a1b21705564687a Add new API for inserting multiple operations into the `DAGCircuit` in Rust. (#13335) * Initial: Add `DAGCircuitConcat` struct and new insertion API. - Add struct that allows continuous addition of instructions into the back of the `DAGCircuit` b... push web-flow github
88.27
See All Builds (13861)