Skip to content

Add an exception requiring PauliStringPhasors only in the left circuit argument for contrib.paulistring.move_pauli_strings_into_circuit.#7985

Open
Acciaccatura wants to merge 7 commits intoquantumlib:mainfrom
Acciaccatura:issue7934
Open

Conversation

@Acciaccatura
Copy link
Copy Markdown
Contributor

As the title says.

  • No tests exist for this change (AFAIK), perhaps we should add one? If so, I may need some guidance on what to test for as my QC knowledge is lacking.
  • Same with adding a docstring - shall we explicitly state what this function is doing, and/or what it requires, or is it OK to leave it as-is?

Fixes #7934

@Acciaccatura Acciaccatura requested review from a team and vtomole as code owners March 29, 2026 23:43
@github-actions github-actions Bot added the Size: XS <10 lines changed label Mar 29, 2026
@pavoljuhas pavoljuhas self-requested a review April 1, 2026 19:55
@github-actions github-actions Bot added size: S 10< lines changed <50 and removed Size: XS <10 lines changed labels Apr 10, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.63%. Comparing base (096fe82) to head (b62cda4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7985      +/-   ##
==========================================
- Coverage   99.63%   99.63%   -0.01%     
==========================================
  Files        1110     1110              
  Lines       99755    99771      +16     
==========================================
+ Hits        99394    99409      +15     
- Misses        361      362       +1     

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

@pavoljuhas pavoljuhas self-assigned this Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AttributeError: 'SingleQubitPauliStringGateOperation' object has no attribute 'pauli_string' in move_pauli_strings_into_circuit

3 participants