Qiskit / qiskit / 8894572858

89%

Build Type

push

github

Committed by web-flow

Commit Message

Support standalone `expr.Var` in QPY (#11651)

* Support standalone `expr.Var` in QPY

This necessitates adding some extra definitions to the circuit header.
When standalone variables are used, we do not re-encode the entire
variable (which would take a miniumum of ~20 bytes per variable), but
instead just emit an index into the order that the variables were
defined at the top of the circuit.

At present, each control-flow scope will store the `Var` nodes anew, so
it's not a perfect only-store-once system.  This is consistent with how
registers are handled, though, and the QPY format isn't particularly
memory optimised as things stand.

* Add backwards compatibility tests

Run Details

124 of 128 new or added lines in 7 files covered. (96.88%)

1 existing line in 1 file now uncovered.

61087 of 68273 relevant lines covered (89.47%)

218624.94 hits per line