Skip to content

Fix ordering in Engine.run_batch#8060

Open
dstrain115 wants to merge 2 commits intoquantumlib:mainfrom
dstrain115:multi_circuit_sweep
Open

Fix ordering in Engine.run_batch#8060
dstrain115 wants to merge 2 commits intoquantumlib:mainfrom
dstrain115:multi_circuit_sweep

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

  • ProcessorSampler incorrectly reconstructs the results from Quantum Engine. It incorrectly assumes that the engine returns results interleaved by sweep point (e.g., S1_P1, S1_P2, S2_P1, S2_P2), when the engine actually returns them grouped by program (e.g., P1_S1, P1_S2, P2_S1, P2_S2).
  • This causes the results to be returned incorrectly if jobs_per_batch>1.

- ProcessorSampler incorrectly reconstructs the results from Quantum Engine.
  It incorrectly assumes that the engine returns results interleaved by sweep point
  (e.g., S1_P1, S1_P2, S2_P1, S2_P2), when the engine actually
  returns them grouped by program (e.g., P1_S1, P1_S2, P2_S1, P2_S2).
- This causes the results to be returned incorrectly if
  jobs_per_batch>1.
@github-actions github-actions Bot added the size: M 50< lines changed <250 label Apr 24, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8060   +/-   ##
=======================================
  Coverage   99.63%   99.63%           
=======================================
  Files        1110     1110           
  Lines       99934    99954   +20     
=======================================
+ Hits        99572    99592   +20     
  Misses        362      362           

☔ 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 marked this pull request as ready for review April 24, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: M 50< lines changed <250

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant