Sequences with different frequencies are not separated

[jevillegasdatTII]

qibolab/src/qibolab/pulses.py

Line 1586 in 898fa86

def separate_overlapping_pulses(self): # -> dict((int,int): PulseSequence):

When sending two pulse sequences over the same channel, provided they don't overlap in time, they are not getting separated into different sequencers. Consequently, the frequency of the first pulse defines the nco frequency for the sequence and all the following pulses use the same frequency irrespective of their settings. For instance, the following code generates the attached sequences sent to qblox.

sequence = PulseSequence()
rx_pulse_q0 = platform.create_RX_pulse(0, start = 0)
rx_pulse_q1 = platform.create_RX_pulse(1, start = 0)
sequence.add(rx_pulse_q0)
sequence.add(rx_pulse_q1)
cr_pulse = platform.create_RX_pulse(1, start = rx_pulse_q1.finish)
cr_pulse.frequency = rx_pulse_q0.frequency
cr_pulse.duration = 1000
sequence.add(cr_pulse)

Z_qcm_rf0_sequencer1_sequence.json
Z_qcm_rf0_sequencer0_sequence.json
Z_qcm_rf0_snapshot.json

Notably, only two sequencers are created, where we need three (channel 2 with two different frequencies and channel one a single one).

[jevillegasdatTII]

Possble fix in commit 94bed20

[hay-k]

@jevillegasdatTII I will take a look at your commits, but for the future can you please not push any commits directly to the branch qblox-ad-hoc-features, and instead create a PR pointing to that branch.

[jevillegasdatTII]

Absolutely, sorry. :(