sandialabs / LoQS / 42695555994

90%

push

github

Fix `loqs-migrate` aliased imports and `PyGSTiPhysicalCircuit` label read-back (#174)

* Keep module aliases when loqs-migrate renames a name

loqs-migrate and the frozen-function decode path no longer write code
that raises NameError for a renamed name reached through an aliased
module import or imported both as a module and by name (#161).

- Do same-module renames in a new _SameModuleRenamer, which keeps the
  user's module alias, `as` name and import placement.
- Re-add the from-import RenameCommand drops for a module-changing
  rename when the file also imports the old module.
- Minor: tests for the alias and mixed-import cases, an updated
  expected string for a multi-name from-import, the module docstring's
  quirks list, and a CHANGELOG entry.

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

* Read saved pyGSTi circuits back on the right qubits

Reading a saved PyGSTiPhysicalCircuit no longer fails or moves gates
to the wrong qubit for integer or prefix-colliding labels (#166).

- Swap labels for parser-safe names with one regex that matches only
  where a label stands, longest first and on its string form.
- Map labels back with a new _map_circuit_labels helper, which keeps
  gate arguments and times; map_qubit_labels_inplace uses it too.
- Minor: a label mismatch raises ValueError instead of an assert, a
  file with no qubits reads back, round-trip and mapping tests, and a
  CHANGELOG entry.

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>

3805 of 4528 branches covered (84.03%)

Branch coverage included in aggregate %.

12350 of 13546 relevant lines covered (91.17%)

0.91 hits per line