Skip to content

WIP: CPTP instruments #693

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
rileyjmurray wants to merge 10 commits into
base: develop
Choose a base branch
from
Draft

WIP: CPTP instruments #693

rileyjmurray wants to merge 10 commits into from
+630 −220

Conversation

@rileyjmurray
Copy link
Contributor

@rileyjmurray rileyjmurray commented Dec 3, 2025 •
edited
Loading

Todo:

  • Docs.
  • Unit tests.
  • Move the two linear operators defined in modelmembers/instruments/cptpinstrument.py into separate files in modelmembers/operations/*.py.
  • Actually run pygsti's normal test suite. CI tests show that my changes broke something. (Shouldn't be hard to fix.)

@rileyjmurray
documentation for DenseOperator
c260399
@rileyjmurray
add the ability to specify custom germs in GSTModelPack.create_gst_ex…
e9bc2a3 
…periment_design
@rileyjmurray
update matrixtools.eigenvalues and matrixtools.eigendecomposition to …
c5a288f 
…allow ndarrays where flags.writeable is False
@rileyjmurray
fix whitespace
9d8e893
@rileyjmurray
optools.py: add minimal_kraus_decomposition function (important for t…
a501a15 
…his PR), rootconj_superop (important for this PR), plus partial_trace and trace_effect (nice to have but not strictly necessary for this PR).
@rileyjmurray
add LinearOperator._kraus_operators function with default implementation
c1d2be0
@rileyjmurray
address warning from not specifying default kwarg in np.linalg.lstsq
172a35e
@rileyjmurray
change Instrument.acton so it no longer requires a TP basis; add supe…
85ae656 
…rket_trace to optools.py
@rileyjmurray
CPTPLND instruments
fe63e73
@rileyjmurray
update Instruments.ipynb advanced example notebook
ab3d7f3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kmrudin kmrudin Awaiting requested review from kmrudin kmrudin will be requested when the pull request is marked ready for review kmrudin is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rileyjmurray