Qiskit / qiskit / 10687317978

90%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix extraction of controlled parametric gates (#13067) (#13080)

The `mutable` check in the controlled-gate `OperationFromPython`
extraction logic to check for a mutated `base_gate` was overzealous,
and would return false positives for parametric controlled gates.  The
only modification to `base_gate` of a standard-library gate that would
not have caused data-model problems from Python space would be setting
the base-gate label, which is used for a public feature of the circuit
visualisers.

The change to `get_standard_gate_name_mapping` is just a minor
convenience to make the gate objects directly appendable to a circuit;
previously, each `Parameter` object was distinct and had a UUID clash
with others of the same name, so could not be used together.  The new
behaviour is purely a convenience for tests; it largely should not be
useful for users to directly append these gates.

(cherry picked from commit 8f3308475)

Co-authored-by: Jake Lishman <jake.lishman@ibm.com>

Run Details

10 of 10 new or added lines in 2 files covered. (100.0%)

7 existing lines in 3 files now uncovered.

66921 of 74493 relevant lines covered (89.84%)

228101.15 hits per line