Qiskit / qiskit / 16508029389

88%

Build Type

push

github

Committed by web-flow

Commit Message

Hide Python dependency in `ConsolidateBlocks` (#14748)

* Remove `py` token from `get_matrix_from_inst`.
- Use `Python::with_gil` in exclusive cases where a `PyGate` is needed. Which is impossible to obtain directly from Rust.
- Use `Python::with_gil` in exclusive cases where the block has more than qargs. A comment needs to be added to explain this behavior (we should not be able to add blocks with more than 2 qubits).

* Docs: Document rust-side `run_consolidate_blocks`.

* Add: Wrapper function exposed to Rust.
- Add wrapper function that exposes the same method but doesn't allow for the user to provide blocks or runs. The previous docstring is now assigned to the rust native wrapper and the warning has been removed.
- Make the python exclusive version of the method private.

* Update crates/transpiler/src/passes/consolidate_blocks.rs

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* Remove: Rust entry point

---------

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

Run Details

32 of 32 new or added lines in 2 files covered. (100.0%)

8 existing lines in 2 files now uncovered.

81433 of 92799 relevant lines covered (87.75%)

546931.93 hits per line