Skip to content

Add GHZ fidelity tool#7944

Merged
eliottrosenberg merged 18 commits intomainfrom
u/eliottrosenbrg/ghz_fidelity
Mar 13, 2026
Merged

Add GHZ fidelity tool#7944
eliottrosenberg merged 18 commits intomainfrom
u/eliottrosenbrg/ghz_fidelity

Conversation

@eliottrosenberg
Copy link
Collaborator

@eliottrosenberg eliottrosenberg commented Mar 9, 2026

A tool for measuring the fidelity of GHZ states.

Utilizes the pauli_string_measurement_with_readout_mitigation tool by @ddddddanni. FYI @shashwatk1998

@eliottrosenberg eliottrosenberg requested review from a team, mrwojtek and vtomole as code owners March 9, 2026 23:05
@eliottrosenberg eliottrosenberg requested a review from maffoo March 9, 2026 23:05
@github-actions github-actions bot added the size: M 50< lines changed <250 label Mar 9, 2026
@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.63%. Comparing base (9dc6526) to head (ee77bed).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7944   +/-   ##
=======================================
  Coverage   99.63%   99.63%           
=======================================
  Files        1108     1110    +2     
  Lines       99584    99662   +78     
=======================================
+ Hits        99218    99296   +78     
  Misses        366      366           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added size: L 250< lines changed <1000 and removed size: M 50< lines changed <250 labels Mar 12, 2026
Copy link
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a couple of small comments I'd like to address before the merge, otherwise LGTM.

Thank you for adding this and for rearranging the ghz code!

PS: I also pushed in a tiny tweak to type annotations.

And cover both mitigated and unmitigated variants.
Use more general Sequence type which is covariant, ie, allowing
subclass-type items.
Copy link
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after speeding up the test

@eliottrosenberg eliottrosenberg added this pull request to the merge queue Mar 13, 2026
Merged via the queue into main with commit 8aacb64 Mar 13, 2026
43 checks passed
@eliottrosenberg eliottrosenberg deleted the u/eliottrosenbrg/ghz_fidelity branch March 13, 2026 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: L 250< lines changed <1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants