Qiskit / qiskit / 10799513349

88%

Build Type

push

github

Committed by web-flow

Commit Message

Oxidize commutative cancellation (#13091)

* fix

* fmt

* comments from code review

* comments from code review

* Update lib.rs

* Apply suggestions from code review

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

* code review

* Fix rustfmt

* Don't change interface for DAGCircuit

Previously this PR was making the op_names field of the DAGCircuit
struct public so it was accessible to the new transpiler pass code.
However, this opened up the possibility of mutating the field by
mistake, instead this makes the field private again and adds a no-copy
method to get an immutable reference to the field. Additionally, there
were some interface changes made to one method in DAGCircuit that were
not needed anymore, which this reverts to minimize the diff.

* Remove last .first().unwrap()

---------

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

Run Details

170 of 190 new or added lines in 7 files covered. (89.47%)

27 existing lines in 5 files now uncovered.

73092 of 82002 relevant lines covered (89.13%)

370019.84 hits per line