-
Notifications
You must be signed in to change notification settings - Fork 6
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
Updates in Cross Resonance calibration routines #987
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## cross_resonance #987 +/- ##
===================================================
- Coverage 97.30% 96.02% -1.28%
===================================================
Files 112 115 +3
Lines 8194 8402 +208
===================================================
+ Hits 7973 8068 +95
- Misses 221 334 +113
Flags with carried forward coverage won't be shown. Click here to find out more.
|
In none of the routines using sweepers, the readout pulse is swept at the same as the CR drive pulse duration, similar to what was discussed in #qiboab/919. I'm not sure how can I do this correctly, but since the typical tests I was running re ~400 ns, this wasn't a problem yet. |
@jevillegasdatTII there is an ongoing PR, qiboteam/qibolab#993, realized by @hay-k, which is restoring some expected Qblox behavior on top of a Qibolab version compatible with the current Qibocal. Did you try the corresponding branch https://github.com/qiboteam/qibolab/tree/qblox-ad-hoc-features? |
I've tested the branch against qiboteam/qibolab@898fa86 and the routines are working well. I found an error though in one routine so I have to cancel the pull request and make it again. No fitting has been implemented in any of the calibration routines. Here are some of the results from today on QPU88 (qiboteam/qibolab_platforms_qrc@4ec5243): Cross Resonance - id: cross resonance sequences
operation: cross_resonance_sequences
targets: [[0,1]]
parameters:
pulse_duration_start: 4
pulse_duration_end: 800
pulse_duration_step: 2
pulse_amplitude: 1.0
nshots: 4000 Cross Resonance Sequences - id: cross resonance sequences
operation: cross_resonance_sequences
targets: [[0,1]]
parameters:
pulse_duration_start: 4
pulse_duration_end: 800
pulse_duration_step: 2
pulse_amplitude: 1.0
nshots: 4000 Chevron - id: cr chevron
operation: cross_resonance_chevron
targets: [[0,1]]
parameters:
pulse_duration_start: 4
pulse_duration_end: 400
pulse_duration_step: 10
amplitude_min_factor: 0.01
amplitude_max_factor: 1.0
amplitude_step_factor: 0.01
pulse_amplitude: 1.0
nshots: 200 Chevron Frequency- id: cr chevron frequency
operation: cross_resonance_chevron_frequency
targets: [[0,1]]
parameters:
pulse_duration_start: 4
pulse_duration_end: 400
pulse_duration_step: 10
freq_width: 50_000_000
freq_step: 1_000_000
pulse_amplitude: 1.0
nshots: 200 |
Follow up on #821 and PR #860
Checklist:
master
main
main