Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QM driver for 0.2 #874

Closed
wants to merge 234 commits into from
Closed

QM driver for 0.2 #874

wants to merge 234 commits into from

Conversation

stavros11
Copy link
Member

@stavros11 stavros11 commented Apr 20, 2024

Updates QM driver to work with the current changes in the 0.2 branch. I got rid of QMPulse and QMSequence since qibolab objects are now simple enough to be used directly. This is not the final version, further changes will be required after the channel refactoring as pulse.qubit and pulse.channel will be dropped.

With the current state it works for the single shot classification routine: http://login.qrccluster.com:9000/3AJ4OvU_QQaTSJdbxk6uwg==
(requires qiboteam/qibolab_platforms_qrc#132 and qiboteam/qibocal#809)

TODO:

  • Test frequency, amplitude and phase sweepers.
  • Test duration sweepers (Rabi length, T1, Ramsey).
  • Unrolling fails in single shot.
  • Refactor QMConfig.
  • Baking.

@stavros11 stavros11 added the qm label Apr 20, 2024
@stavros11 stavros11 added this to the Qibolab 0.2.0 milestone Apr 20, 2024
Copy link

codecov bot commented Apr 20, 2024

Codecov Report

Attention: Patch coverage is 28.78788% with 141 lines in your changes are missing coverage. Please review.

Project coverage is 49.10%. Comparing base (f829e8f) to head (a50962f).

❗ Current head a50962f differs from pull request most recent head 7116fd2. Consider uploading reports for the commit 7116fd2 to get more accurate results

Files Patch % Lines
src/qibolab/instruments/qm/config.py 5.17% 55 Missing ⚠️
src/qibolab/instruments/qm/controller.py 23.52% 26 Missing ⚠️
src/qibolab/instruments/qm/acquisition.py 48.88% 23 Missing ⚠️
src/qibolab/instruments/qm/program.py 39.47% 23 Missing ⚠️
src/qibolab/instruments/qm/sweepers.py 39.13% 14 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              0.2     #874      +/-   ##
==========================================
+ Coverage   48.75%   49.10%   +0.35%     
==========================================
  Files          61       61              
  Lines        5532     5380     -152     
==========================================
- Hits         2697     2642      -55     
+ Misses       2835     2738      -97     
Flag Coverage Δ
unittests 49.10% <28.78%> (+0.35%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@stavros11 stavros11 changed the title Qmdelay QM driver for 0.2 Apr 20, 2024
@stavros11 stavros11 marked this pull request as draft April 20, 2024 20:48
@stavros11
Copy link
Member Author

Superseded by #933.

@stavros11 stavros11 closed this Jul 13, 2024
@stavros11 stavros11 deleted the qmdelay branch July 13, 2024 20:13
@stavros11 stavros11 restored the qmdelay branch July 13, 2024 20:14
@stavros11 stavros11 deleted the qmdelay branch July 13, 2024 20:15
@alecandido alecandido mentioned this pull request Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants