Fix Version/s: None
I was inconsistent with writing round-trip tests for calibrations, and this ticket is the clean-up to fix that.
Christopher Waters added a comment - - edited
Christopher Waters added a comment - - edited https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33946/pipeline https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33991/pipeline
Andrés Alejandro Plazas Malagón added a comment -
Looks good, just two minor questions asking for clarification.
Andrés Alejandro Plazas Malagón added a comment - Looks good, just two minor questions asking for clarification.
Only crosstalk was found missing a round-trip testing case. That's been fixed now.
This also fixes the problems those tests identified in the crosstalk calibration.
I've also updated the IsrCalib._eq_ method to reduce code duplication.