This is an idea from @silviamich and me after we faced issues with units in tod operations.
We could force all tod computations (scan_map, fill_tod_with_hwp_harmonics, add_2f, add_non_linearity, etc.) to have compatible units. I am thinking of something similar (or exactly the same) as what is done in the map operations currently. This would avoid e.g. adding differential emissions in Kelvins to a TOD in spectral radiance units. Every operation changing the tod of an observation would start by passing a unit check.
We can discuss this better in the next meeting, meanwhile I will try to come up with a draft PR to do this.
This is an idea from @silviamich and me after we faced issues with units in tod operations.
We could force all tod computations (scan_map, fill_tod_with_hwp_harmonics, add_2f, add_non_linearity, etc.) to have compatible units. I am thinking of something similar (or exactly the same) as what is done in the map operations currently. This would avoid e.g. adding differential emissions in Kelvins to a TOD in spectral radiance units. Every operation changing the tod of an observation would start by passing a unit check.
We can discuss this better in the next meeting, meanwhile I will try to come up with a draft PR to do this.