Qiskit / qiskit / 16153639797

88%

Build Type

push

github

Committed by web-flow

Commit Message

Remove py token from the dag_to_circuit signature (#14707)

This commit remove the py token from the dag_to_circuit signature. This
enables calling this function from a standalone C context. The python
usage in the function only comes into play if there are Python defined
operations and we need to deepcopy them, or if there are any
ParameterExpression for an instruction which needs to clone it. Since
neither ParameterExpression or Python defined objects can exist in C we
can avoid the py usage in those paths. For the ParameterExpression path
since we have the py-clone feature enabled we can simply just replace
the clone_ref() call with a clone() call. This has extra overhead
equivalent to acquiring the gil explicitly but avoids the explicit
python usage. When we have ParameterExpression in Rust this usage
will continue to work because it will clone the rust object instead
of dispatching to Python under the covers.

Run Details

9 of 9 new or added lines in 3 files covered. (100.0%)

22 existing lines in 4 files now uncovered.

81460 of 92829 relevant lines covered (87.75%)

548753.3 hits per line