Skip to content

Conversation

dstrain115
Copy link
Collaborator

  • when calling merge_single_qubit_gates_to_phxz_symbolized, if remaining symbols is empty, then an empty ListSweep is added to the end of the sweep.
  • This empty ListSweep confuses parsers, which then interpret this as a sweep of zero length and return an empty list.
  • This now only adds this ListSweep if there is something to add.

- when calling merge_single_qubit_gates_to_phxz_symbolized, if remaining
  symbols is empty, then an empty ListSweep is added to the end of the
  sweep.
- This empty ListSweep confuses parsers, which then interpret this as a
  sweep of zero length and return an empty list.
- This now only adds this ListSweep if there is something to add.
@dstrain115 dstrain115 requested review from vtomole and a team as code owners September 24, 2025 17:51
@github-actions github-actions bot added the size: S 10< lines changed <50 label Sep 24, 2025
Copy link

codecov bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.37%. Comparing base (23afd92) to head (04a6ff4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7664   +/-   ##
=======================================
  Coverage   99.37%   99.37%           
=======================================
  Files        1082     1082           
  Lines       96689    96691    +2     
=======================================
+ Hits        96082    96088    +6     
+ Misses        607      603    -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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dstrain115 dstrain115 added this pull request to the merge queue Sep 24, 2025
Merged via the queue into quantumlib:main with commit b1f8b82 Sep 24, 2025
35 checks passed
@dstrain115 dstrain115 deleted the merged_listsweep branch September 24, 2025 19:17
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.

2 participants