Qiskit / qiskit / 14045064185

88%

Build Type

push

github

Committed by web-flow

Commit Message

Remove Python dependency for creating a blank dag (#14069)

* Remove Python dependency for creating a blank dag

This commit removes the py token argument for the dag constructor which
wasn't strictly needed. The only pieces of the dagcircuit struct which
need python is the metadata atribute (which is an `Option<PyDict>`) and
the var tracking sets. This commit rearranges the constructor so it
doesn't actively need python. This is achieved by moving the var
tracking sets inside a once lock which is only initialized in a context
where vars are being used. The metadata is changed to None by default.

The other use case was the control flow module
which wasn't strictly required and is removed in this PR and using the
import once cell mechanism that we use for this everywhere else.

* Remove conditional import of OnceLock since it's always used

* Update rust tests

* Remove _bound suffix from method names

* Add docstring to VarsByType

Run Details

131 of 133 new or added lines in 3 files covered. (98.5%)

5 existing lines in 3 files now uncovered.

72642 of 82473 relevant lines covered (88.08%)

371497.84 hits per line