Qiskit / qiskit / 15045660902

88%

Build Type

push

github

Committed by web-flow

Commit Message

Avoid unecessary cloning in disjoint layout Python path (#14366)

In the recently merged #14177 the python path for running the disjoint
components was unecessarily copying the input dag before running the
inner layout algorithm in the case there were no connected components to
run. This was resulting in extra overhead in the nightly benchmarks for
the DenseLayout pass when there were was no disjoint components in the
connectivity graph. This is because the clone of the dag was relatively
expensive relative to the overall cost of the pass. Since we can work
by reference by handling borrowing the original input dag from Python
by being a bit more explicit with the typing to make it explicit the
Python entrypoint is working with Python and with some careful scoping
to avoid runtime borrow checking issues. This commit applies these
changes and removes the dag clone in the case when there is no disjoint
handling needed.

Run Details

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

11 existing lines in 4 files now uncovered.

76379 of 86105 relevant lines covered (88.7%)

427422.27 hits per line