Qiskit / qiskit / 17290672001

88%

Build Type

push

github

Committed by web-flow

Commit Message

C API: Burn `Optimize1qGatesDecomposition`. (#14710)

* Initial: Introduce C-API Access to `Optimize1qGatesDecomposition`.
- The following commit adds initial access to a standalone C foreign function interface that allows users to run the `Optimize1qGatesDecomposition` transpiler pass. The feature as it stands requires the user to build a target prior to submitting the circuit; This might change in the future.
- Added 1 initial testing suite for the pass which converts a sequence of H gates into a different sequences depending on the provided target. More tests will be added with time.

* Fix: Add target identity test.

* Chore: Fix docstring and free some pointers.

* Add: Support for optional target for `Optimize1qGatesDecomposition` C API.
- Add ability of running the pass without passing a `Target`.
- Add proper formatting to unit tests with multiple tests inside.
- Add tests without target.

* Chore: Add release note.

* Chore: Fix docstring.

* Chore: Fix bug in release note

* Chore: Address review comments
- Make tha function call run in-place and modify the pointer to the `QkCircuit` in use.
- Remove `h_p_target` from tests.
- Use parametric gates in the target with generic parameter expressions instead of fixed ones.

* Update crates/cext/src/transpiler/passes/optimize_1q_decomposition.rs

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

* Chore: Fix docstring

* Chore: Fix release note

* Update test/c/test_optimize_1q_decomposition.c

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Chore: Address review comments.

* Fix: Review comments

* Fix: Address some memory leakage.

* Fix: Incorrect function name call

* Chore: Address different review items
- Rename files from `optimize_1q_decomposition` to `optiize_1q_sequences`.
- Fixed previous oversight that counted operations before the pass was ran.

* Fix: Replace `QkOpCounts` failures until it is fixed.

* Fix: Add final review comments.
- Update release note to include explan... (continued)

Run Details

17 of 17 new or added lines in 1 file covered. (100.0%)

17 existing lines in 4 files now uncovered.

90331 of 102179 relevant lines covered (88.4%)

486389.25 hits per line