Qiskit / qiskit / 19740043964

88%

Build Type

push

github

Committed by web-flow

Commit Message

Unified commutative optimization (#15047)

* Placeholder for new python and rust passes

* preliminary implementation of some of the components

* copying + slightly adapting CC tests

* Next batch of improvements.

* cleanup

* more cleanup

* docstrings

* adapting tests from CommutativeInverseCancellation; fixes

* getting rid of py token

* improving return types to avoid unwrap

* hanlding the case when two pauli evolution gates cannot be merges + test

* code sharing & deduplication

* improvements and fixes

* undoing let chains rewrite (CI does not like it)

* running black

* partially addressing review comments

* renaming matrix_use_view_only -> matrix_from_definition

* docstring comment

* Update, and most importantly this adds @cryoris as a coathor

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

* implementing Julien's suggestion

* update after merge conflict

* Improved comparison for SparsePauliOps + updating the test

* black

* simplifying code based on review

* handling CU1 in RemoveIdentityEquivs and CommutativeOptimization

* further support for CU1

* updating tests

* parametric rx test

* moving argument-based precheck earlier in the commutation flow

* micro-optimization: avoiding to compute op.view() when not required

* revamping an older test

* Revert "revamping an older test"

This reverts commit 69cc0dcf8.

* changing default values for matrix-based computations to 0

* update tests with custom gates that require matrix manipulations

* checking whether two instructions are disjoint upfront

* improve return types of can_remove and is_identity_equiv

* renaming can_remove to a more meaningful name

* reverting the unintended change to get_matrix in remove_identity_equivs pass

* Checking that the gate's name is PauliEvolution before calling python

* adding leading underscore to internal function, improving docstringss

* improve the argument ordering and the docstring for get_matrix m... (continued)

Run Details

431 of 474 new or added lines in 14 files covered. (90.93%)

27 existing lines in 4 files now uncovered.

95651 of 108247 relevant lines covered (88.36%)

1194998.56 hits per line