Skip to content

Fix/variant removal#11

Closed
Bloooman wants to merge 6 commits intoQuantumSavory:masterfrom
Bloooman:fix/variant_removal
Closed

Fix/variant removal#11
Bloooman wants to merge 6 commits intoQuantumSavory:masterfrom
Bloooman:fix/variant_removal

Conversation

@Bloooman
Copy link

Context: CiruitOp.Pauli is unnecessary
Summary:
Removed CircuitOp.Pauli from all tests and replace them with CircuitOp.ExpQuatPiPauli if a vairant is necessary for the test
PBCCompiler tests passed after removal
Question: WHat's the purpose of CircuitOp.Prep_Magic? I removed them at this point as they do not fit in my current logic implementation

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.94%. Comparing base (ee5c4cc) to head (d7665a0).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   78.26%   78.94%   +0.68%     
==========================================
  Files           4        4              
  Lines         184      171      -13     
==========================================
- Hits          144      135       -9     
+ Misses         40       36       -4     

☔ 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.

@Bloooman Bloooman closed this Feb 20, 2026
@Bloooman Bloooman reopened this Feb 20, 2026
@Bloooman Bloooman force-pushed the fix/variant_removal branch from 4edb33b to d7665a0 Compare February 20, 2026 21:27
@Krastanov
Copy link
Member

Could you elaborate why this change is useful?

@Bloooman Bloooman closed this Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants