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-13137-4e6a874ac7d2b4736e8e87546f023b5592fc059c
branch: gh-readonly-queue/main/pr-13137-4e6a874ac7d2b4736e8e87546f023b5592fc059c
CHANGE BRANCH
x

coverage: 88.872% (+0.02%) from 88.85%
11076119993

push

github

web-flow 
[Oxidize BasisTranslator] Add rust-native `compose_transforms()` (#13137)

* Initial: Add `compose_transforms` and `get_example_gates` to Rust.

* Add: `compose_transform` logic in rust

* Fix: Correct the behavior of `compose_transforms`.
-  Use `PackedOperation.control_flow` instead of `CONTROL_FLOW_OP_NAMES` to check if an instructuion is a control flow operation.
- Remove panic statement when checking for example gates.
- Use qreg when adding an instruction to the mock_dag in `compose_transforms`.
- Add missing comparison check in for loop to compare the mapped instructions.
- Use borrowed `DAGCircuit` instances in the recursion of `get_example_gates`, do not use extract.
- Fix behavior of `get_example_gates` to return `PackedInstruction` instances instead of `NodeIndex`.

* Fix: Leverage rust-based `circuit_to_dag` converters.
- Use `circuit_to_dag` and `DAGCircuit::from_circuit_data` to convert `QuantumCircuit` instances.

* Format: Fix indentation of import lines in `compose_transforms.rs`

* Formatting: Separate complicated type aliases

* Fix: Adapt to new `DAGCircuit` limitations

* Format: Remove unused imports in BasisTranslator

* Fix: Adapt to #13143

* Fix: Code review comments
- Remove unused `circuit_to_dag` and `OperationRef` imports.
- Streamline complex types into simpler aliases.
- Rename `CircuitRep` to `CircuitFromPython`.
- Reshape `get_example_gates` to work with `CircuitData` instances during its recursion.
- Remove unnecessary clone of `gate_name` in `compose_transform`.
- Use `mapped_instructions` values when iterating in `compose_transforms`.
- Rename `DAGCircuit::substitute_node_with_dag` to `DAGCircuit::py_*` in rust.
- Adapted `_basis_search` to return a tuple of tuples instead of a 4-tuple.

* Fix: More commments from code review
- Remove stale comment related to #3947
- Rename tuple instances to `GateIdentifier`.
- Rename `doomed_nodes` to `nodes_to_replace`.
- Add docstring for `get_example_gates`.
- Rename `get_exam... (continued)

160 of 164 new or added lines in 8 files covered. (97.56%)

17 existing lines in 4 files now uncovered.

74136 of 83419 relevant lines covered (88.87%)

375345.4 hits per line

Relevant lines Covered
83419 RELEVANT LINES 74136 COVERED LINES
375345.4 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13137-4e6a874ac7d2b4736e8e87546f023b5592fc059c

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11076119993 gh-readonly-queue/main/pr-13137-4e6a874ac7d2b4736e8e87546f023b5592fc059c [Oxidize BasisTranslator] Add rust-native `compose_transforms()` (#13137) * Initial: Add `compose_transforms` and `get_example_gates` to Rust. * Add: `compose_transform` logic in rust * Fix: Correct the behavior of `compose_transforms`. - Use ... push web-flow github
88.87
See All Builds (9023)