Qiskit / qiskit / 11572090796

88%

Build Type

push

github

Committed by web-flow

Commit Message

Add RemoveIdentityEquivalent transpiler pass (#12384)

* add DropNegligible transpiler pass

* use math.isclose instead of numpy.allclose

* use pass directly instead of constructing pass manager

* lint

* allow passing additional gate types

* add docstrings to test classes

* Generalize the pass for all gates and rewrite in rust

This pivots the pass to work for all gates by checking all the
parameters in the standard gate library are within the specified
tolerance, and the matrix is equivalent to an identity for any other
gate defined in Python (with a matrix defined). To improve the
performance of the pass it is written in rust now. Additionally the
class is named to RemoveIdentityEquivalent to make the purpose of the
pass slightly more clear.

* Don't remove zero qubit gates

* Ensure computed gate error is always positive

* Pass docstring improvements

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

* Move test file

* Fix lint

* Fix docs

* Add optimized identity check for Pauli gates

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

* Mention zero qubit gates in the docstring

* Update approximation degree logic in absense of target

* Expand test coverage and fix bugs

---------

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>
Co-authored-by: Julien Gacon <gaconju@gmail.com>

Run Details

98 of 106 new or added lines in 6 files covered. (92.45%)

6 existing lines in 2 files now uncovered.

74997 of 84566 relevant lines covered (88.68%)

365199.55 hits per line