Qiskit / qiskit / 26091433602

88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix `InverseCancellation` for symmetric pairs (#16153)

* Fix `InverseCancellation` for symmetric pairs.

Symmetric gates should get cancelled irrespective of the qubit order. This did not happen in InverseCancellation, which didn't consistently cancel CS and CSdg gates in the default setup since their qubit argument were out of order. Coincidentally _it did_ cancel these gates in a specific order as a by product since the boolean logic was not grouped correctly, as reported in #15855. This commit fixes this behavior.

This commit does not more generally enable handling of symmetric, user-supplied gates.

* Better inverse pair structure

Co-authored-by: Jake Lishman <jake.lishman@ibm.com>

* More precise wording in reno

---------

Co-authored-by: Jake Lishman <jake.lishman@ibm.com>

Coverage Stats

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

24 existing lines in 3 files now uncovered.

107900 of 123052 relevant lines covered (87.69%)

950074.85 hits per line