sandialabs / pyGSTi

A python implementation of Gate Set Tomography
http://www.pygsti.info
Apache License 2.0
132 stars 55 forks source link

Bugfix 0.9.12.2 #421

Closed sserita closed 2 months ago

sserita commented 2 months ago

This PR contains three minor bugfixes slated for 0.9.12.2:

  1. Tutorial updates for #317
  2. A revamp of the leakage example notebook following broken tests from PR #418
  3. Additional labels to indicate report figure/table titles can be clicked for #416

The leakage notebook may not be in a completely finished state. These updates are more or less making it so that circuit line labels are explicit, but it's a little awkward/still not working completely, especially for the kite structure example at the bottom. Suggestion now are welcome, or we can just slate it reworking later.

coreyostrove commented 2 months ago

Piggybacking on this PR with a bugfix for some reporting code I broke with PR #415 which got caught by the extra tests on beta.

coreyostrove commented 2 months ago

Everything looks good to me for the changes that you had made. There is a substantive changed I pushed piggybacking onto this PR. Whenever you get a chance if you could review that (and assuming there aren't any issues therein) then I think this is good to go.