Qiskit / qiskit / 9416939207

88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix `QuantumCircuit.depth` with zero-operands and `Expr` nodes (#12429)

This causes `QuantumCircuit.depth` to correctly handle cases where a
circuit instruction has zero operands (such as `GlobalPhaseGate`), and
to treat classical bits and real-time variables used inside `Expr`
conditions as part of the depth calculations.  This is in line with
`DAGCircuit`.

This commit still does not add the same `recurse` argument from
`DAGCircuit.depth`, because the arguments for not adding it to
`QuantumCircuit.depth` at the time still hold; there is no clear meaning
to it for general control flow from a user's perspective, and it was
only added to the `DAGCircuit` methods because there it is more of a
proxy for optimising over all possible inner blocks.

Run Details

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

10 existing lines in 2 files now uncovered.

62417 of 69678 relevant lines covered (89.58%)

252778.33 hits per line