Skip to content

Keysight batching support - #1460

Draft
sorewachigauyo wants to merge 1 commit into
mainfrom
keysight-batching
Draft

Keysight batching support#1460
sorewachigauyo wants to merge 1 commit into
mainfrom
keysight-batching

Conversation

@sorewachigauyo

Copy link
Copy Markdown
Contributor

Currently in the keysight driver, there is no preprocessing for the batching, which means the entire list of PulseSequences is passed to the driver for compilation.

Since the 2Q RB has switched to batching, our execution now fails because the compiler cannot resolve such a large sequence.

This PR aims to fix this by introducing a preprocessing step for the batching to cut down the number of sequences per execution.

As a test case, we cut down the time for a 7 pair 2Q RB from 270-280s (no batching/single circuit execution) to 200-210s for the same workload. Though, the time saving is actually larger (actual execution time is about 170s) because the time for the RB sequence generation has increased due to the implementation in qiboteam/qibocal#1490 compared to the circuit reuse in qiboteam/qibocal#1416.

rb-2q-fast.tar.gz

TODO:

  • Work out instrument batching limits

@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.33%. Comparing base (fb080b2) to head (e107912).
⚠️ Report is 82 commits behind head on main.

Files with missing lines Patch % Lines
src/qibolab/_core/instruments/keysight/qcs.py 0.00% 20 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1460      +/-   ##
==========================================
- Coverage   37.45%   37.33%   -0.13%     
==========================================
  Files         114      114              
  Lines        5900     5919      +19     
==========================================
  Hits         2210     2210              
- Misses       3690     3709      +19     
Flag Coverage Δ
unittests 37.33% <0.00%> (-0.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@sorewachigauyo sorewachigauyo added the do not merge Prototype implementation for demonstration purposes label Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Prototype implementation for demonstration purposes drivers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants