Qiskit / qiskit / 7053011598

88%

Build Type

push

github

Committed by web-flow

Commit Message

Track runtime variables in control-flow builders (#10977)

* Track runtime variables in control-flow builders

This adds support for all the new classical runtime variables through
the control-flow builder interface.  In particular, most usefully it
automatically manages the scoping rules for new declarations and inner
variable accesses, and ensures that its built scopes automatically close
over any variables used within them (including making sure nested scopes
do the same thing).

The builder API is factored out a little into an explicit interface
object, with `QuantumCircuit` getting an explicit implementation of
that.  This is done because the number of separate API methods we would
have needed to pass around / infer was getting overly large, and this
allows us to just use standard virtual dispatch to automatically do the
right thing.

Python doesn't have a way to have an object implement an
interface other than by structural (duck) typing, so to avoid name
leakage and collisions, we instead make `QuantumCircuit`'s
implementation a friend class that handles the inner state on its
behalf.

Not everything control-flow-builder related is factored out into the API
because it didn't seem overly useful to do this, especially when the
overridden behaviour would just have been to throw exceptions.

* :us:

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

* More descriptive variable name

* Remove superfluous private statement

---------

Co-authored-by: Matthew Treinish <mtreinish@kortar.org>

Run Details

182 of 188 new or added lines in 2 files covered. (96.81%)

8 existing lines in 2 files now uncovered.

59920 of 68516 relevant lines covered (87.45%)

275489.47 hits per line