Qiskit / qiskit / 14389768512

88%

Build Type

push

github

Committed by web-flow

Commit Message

Remove unecessary nodes field in SabreDAG (#14199)

The SabreDAG struct was carrying a nodes field which was a Vec of the
topologically sorted node descriptions used to create the inner graph
object. The format of this field mirrors what Python passes to Rust when
initially creating the DAG. The reason this was being kept around was
for reversing the graph when working with control flow ops. However, in
practice this isn't necessary because all the data contained in the vec
is also contained in the graph and we can just reverse the graph
directly and save having to carry around the entire object. Also
removing this field simplifies the step of moving the sabre dag
construction into Rust which is required for porting the remainder of
the pass to Rust.

Run Details

17 of 17 new or added lines in 2 files covered. (100.0%)

6 existing lines in 1 file now uncovered.

73732 of 83556 relevant lines covered (88.24%)

436698.13 hits per line