openquantumhardware / qick

QICK: Quantum Instrumentation Control Kit
MIT License
177 stars 74 forks source link

Fixed bug in `QuickProgram.reset_phase()` when default registers are used. #235

Closed hatlabcz closed 4 months ago

hatlabcz commented 4 months ago

When default registers were used, the parameters for playing the dummy pulse that triggers the phase reset were not set properly, which leaded to a non-zero pulse played at reset_phase(). This was because I forgot to also temporarily empty the AbsRegisterManager.default_regs before setting those dummy parameters. This is fixed now.

meeg commented 4 months ago

Looks good, thanks!