Qiskit / qiskit / 14996828508

88%

Build Type

push

github

Committed by web-flow

Commit Message

Remove py token from dag methods (#14313)

* Remove py token from dag methods

This commit updates several of the dag methods to remove the explicit
use of a py token. The active python usage in the dag is pretty minimal
after recent changes, the main places are any interaction with control
flow. As we move towards directly exposing transpiler functionality via
the C API we need to be able to do this in a python free context. Since
control flow is only defined in Python we can move the py token usage to
a with_gil() call on demand when we know we're working with control
flow. The actual python usage was pretty minimal but it ends up getting
wired through a ton of methods because of a few places. By making this
change we can greatly reduce the use of py token in function signatures
which get us much closer to being able to expose transpiler
functionality in C.

* Remove qargs_interner_mut method

* Make DAGCircuit name field Option<String>

* Simplify may_have_additional_wires

* Move gil acquisition to additional_wires callers

* Split bit/reg removal methods between python and rust

Run Details

518 of 565 new or added lines in 10 files covered. (91.68%)

28 existing lines in 3 files now uncovered.

76039 of 85732 relevant lines covered (88.69%)

428932.56 hits per line