Qiskit / qiskit / 14525077100 / 1
88%
main: 88%

LAST BUILD BRANCH: unitary-synthesis
DEFAULT BRANCH: main
Ran
Files 789
Run time 27s
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
coverage: 88.272% (+0.06%) from 88.213%
14525077100.1

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)

73824 of 83632 relevant lines covered (88.27%)

439197.32 hits per line

Source Files on job 14525077100.1