Qiskit / qiskit / 14627759099

88%

Build Type

push

github

Committed by web-flow

Commit Message

Use `OnceLock` to cache a `NormalOperation` in `Target` (#13995)

* Add: Use `OnceLock` to cache a `NormalOperation` in `Target`
- As we plan to use `PackedOperation` to add instructions to a `Target` in Rust. We should not require having a python instance exists within. The following commits fix that by ensuring that a python instruction is not required, and if so it can be built based on a `PackedOperation`'s internal data.
- This should not have any visible effects on the current implementations, but can be proven to work via the added unit-tests.

* Fix: Address review comments.
- Remove `From` implementations in favor of `from_packed_operation` in both `TargetOperation`, and `NormalOperation`.

* Test: Mover rust conversion testing to Python

* Update crates/accelerate/src/target_transpiler/mod.rs

Co-authored-by: Kevin Hartman <kevin@hart.mn>

* Fix: Propagate `Result` in `create_py_op`

* Fix: Store a `PyResult` for the cached gate

---------

Co-authored-by: Kevin Hartman <kevin@hart.mn>

Run Details

28 of 66 new or added lines in 1 file covered. (42.42%)

29 existing lines in 2 files now uncovered.

74245 of 84478 relevant lines covered (87.89%)

435693.32 hits per line