Details
-
Type:
Story
-
Status: Won't Fix
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: coadd_utils, jointcal, meas_base
-
Labels:
-
Story Points:8
-
Team:Alert Production
Description
jointcal's PerTractCcdDataIdContainer doesn't have any tests, so its exact behavior given various inputs is not entirely clear. We should add some tests for it using some fake visit/tract combinations: list of visits all in a tract; list of tracts with no visits; list of visits in different tracts, would be a good start.
As part of this, we should explore merging the existing versions of this code into one, so that the code and tests only live in one package (possibly pipe_base? Or maybe daf_butler?). Possibilities to include in this unification include:
- jointcal.dataIds.PerTractCcdDataIdContainer
- meas.base.forcedPhotCcd.PerTractCcdDataIdContainer
- coadd.utils.CoaddDataIdContainer
- pipe.tasks.assembleCoadd.AssembleCoaddDataIdContainer
This should wait for Gen3 Butler, as that will eliminate the "Reading WCS to determine tracts" part of the code, since making up fake WCSs would make writing the test much harder.
Attachments
Issue Links
- is blocked by
-
DM-13391 Develop Gen3 Butler sufficient to support CmdLineTask execution
- Done
I believe we can mark this "won't fix" since it won't be necessary for gen3? Similarly for the other three similar containers listed above.