JAZZ protocol#1449
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1449 +/- ##
==========================================
- Coverage 94.71% 94.24% -0.47%
==========================================
Files 135 136 +1
Lines 10778 10861 +83
==========================================
+ Hits 10208 10236 +28
- Misses 570 625 +55
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
Here is some test data from our QPU We moved the coupler slightly to exaggerate the ZZ effects, but this also changed the dressed frequencies of the qubit pair so the XY pulses for the exaggeration are slightly off. jazz_test.tar.gz |
lballerio
left a comment
lballerio
left a comment
There was a problem hiding this comment.
I agree that this protocol belongs to the two qubit interaction folder, maybe might be also reasonable to move ZZ experiment to the same folder and maybe renaming a bit (not necessary to do here, just a stream of thoughts).
| start_wait: float | ||
| end_wait: float | ||
| nsteps: int |
There was a problem hiding this comment.
as far as I understood we want to transition from this syntax to taking a range tuple, so even though is ugly we could accept either one or the other.
| for channel, pulse in q0_natives.RX(): | ||
| sequence += [(channel, pulse)] | ||
| sequence += [(ro_channel, Delay(duration=pulse.duration))] | ||
| sequence += [(q1_drive_channel, Delay(duration=pulse.duration))] | ||
| # X(pi) on q1 | ||
| for channel, pulse in q1_natives.RX(): | ||
| sequence += [(channel, pulse)] | ||
| sequence += [(ro_channel, Delay(duration=pulse.duration))] | ||
| sequence += [(q0_drive_channel, Delay(duration=pulse.duration))] |
There was a problem hiding this comment.
do we really have at the moment a list with more than 1 element for RX?
| # MZ | ||
| sequence += [(ro_channel, ro_pulse)] |
There was a problem hiding this comment.
maybe we can also measure, save and plot even the q1 state, might be informative
| if len(targets) != 1: | ||
| raise ValueError("simultaneous pair is not supported for JAZZ.") |
There was a problem hiding this comment.
In Ramsey ZZ we decided to open the protocol to multiple QubiPairId and simply check if some qubits are repeated.
Better to choose something consistent for both, whatever the decision is.
3 participants
No description provided.