Closed ctuguinay closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.52%. Comparing base (
9f56124
) to head (1b55517
). Report is 129 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Yeah I agree with this approach. Just much easier to implement.
The only suggestion I have is to do the following right at the top of _compute_cal
, then we don't need to change the arguments. This also makes our intension clear:
# Make waveform_mode "FM" equivalent to "BB"
waveform_mode = "BB" if waveform_mode == "FM" else waveform_mode
@leewujung Thanks! I added your suggestion. This should be ready for another quick review.
Addresses #1334
This was the easiest way I could envision allowing FM. Otherwise, I would've reviewed all the calibrate code and added FM there. What are your thoughts @leewujung?