Closed stavros11 closed 1 month ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 66.82%. Comparing base (
a964ab2
) to head (dd70a1b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Following today's discussion, we could probably merge this for the release tomorrow. I just implemented multiple frequencies on the same qubit for readout as well. I only implemented it for when using Octaves, not external mixers, as this is sufficient for the current qw5q_platinum platform.
The change may not be optimal codewise, but I wouldn't spend more time as long as it works, as this part will be completely refactored in 0.2 (using channel names instead of hardcoding the element names, etc.).
@alecandido @Edoardo-Pedicillo @andrea-pasquale could you please review this asap?
I am not sure if I would merge that because it was a very quick and messy fix, but we should certainly support it on all drivers for 0.2