sandialabs / LoQS / 41183567566

90%

push

github

Make decoded codepack instructions run and compare equal (#167)

* Keep codepack instruction functions free of closures

Two codepack instruction functions read their builder's locals, so
they raised NameError once read back from a file (#151); they now take
those values from Instruction.data, and closures warn at build time.
- Pass the 5Q unflagged lookup table and surf17's stabilizer lists in
  through data, and import pymatching inside the parity function.
- Warn in Instruction when apply_fn or map_qubits_fn closes over
  enclosing-function names and no serialized source was passed.
- Minor: read-back run tests for both decoders, a no-closure guard over
  the single-patch codepacks, a warning test, leakage tutorial fix,
  docstrings, .flake8 entry.

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>
Co-authored-by: claude-opus-5-5 <noreply@anthropic.com>

* Compare physical circuits by content

Instructions compared their physical circuits as text, which pyGSTi's
parser changes by reordering parallel layers on decode, so decoded
codepack instructions compared unequal and a fresh-runner resume
refused (#151); circuits now compare by content.
- Add BasePhysicalCircuit.__eq__ (same class, qubit labels in order,
  backend circuit ==) and a per-class __hash__.
- Minor: per-backend json/h5 round-trip equality, content-inequality,
  reordered-layer and surf17 fresh-runner resume tests; CHANGELOG.

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>
Co-authored-by: claude-opus-5-5 <noreply@anthropic.com>

---------

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>

3670 of 4410 branches covered (83.22%)

Branch coverage included in aggregate %.

11923 of 13130 relevant lines covered (90.81%)

0.91 hits per line