15045660902
88%

Ran 15 May 2025 01:30PM UTC

Jobs 1

Files 809

Run time 2min

Badge

Embed ▾

Committed 15 May 2025 12:27PM UTC coverage: 88.704% (-0.001%) from 88.705%

Build # 15045660902

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.

Coverage Stats

76379 of 86105 relevant lines covered (88.7%)

427422.27 hits per line

Coverage Regressions

Lines	Coverage	∆	File
6	97.61	0.0%	crates/qasm2/src/parse.rs
2	92.73	0.5%	crates/qasm2/src/lex.rs
2	93.94	-1.01%	qiskit/transpiler/passes/layout/sabre_layout.py
1	94.23	-0.23%	crates/qasm2/src/expr.rs

Jobs

ID	Job ID	Ran	Files	Coverage
1	15045660902.1	15 May 2025 01:30PM UTC	809	88.7	GitHub Action Run

Qiskit / qiskit / 15045660902
88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 15045660902

Qiskit / qiskit / 15045660902 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 15045660902

Qiskit / qiskit / 15045660902
88%

README BADGES
x