Qiskit / qiskit / 15593047053

88%

Build Type

push

github

Committed by web-flow

Commit Message

Add more `DAGCircuit::copy_empty_like`-related methods (#14496)

* Add capacity-setting `DAGCircuit::copy_empty_like` constructors

`petgraph` currently does not expose any APIs to expand the capacity of
the `Vec`s backing `Graph` after construction.  This motivates allowing
`copy_empty_like` to accept capacities.

* Add physical-circuit variant of `DAGCircuit::copy_empty_like`

The new `DAGCircuit::physical_empty_like_with_capacity` is similar in
spirit to `copy_empty_like_with_capacity`, except that the output is
defined over a certain number of physical qubits.  This operation is
common (and error-prone) in layout passes, so is centralised here.

* Add `DAGCircuit::qubitless_empty_like_with_capacity`

This is the base method of all the other `*_empty_like_*` methods.  We
don't need the full generality of allowing other classical data to be
present or not; we don't know of any uses of that within the library.

* Restore interner-copying behaviour

* Include comment on Interner copying

* Make `DAGCircuit::name` and `metadata` public

These are logically entirely public from Python space and have no
data-coherence troubles.  This allows Rust to set them as well.

* Add Rust-space test of `physical_empty_like`

Run Details

123 of 134 new or added lines in 1 file covered. (91.79%)

25 existing lines in 4 files now uncovered.

83136 of 94487 relevant lines covered (87.99%)

517212.81 hits per line