Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_algorithms
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:AP F22-5 (October)
-
Team:Alert Production
-
Urgent?:No
Description
Once gen2 has been removed, the base class and configs for the new ConvertReferenceCatalog should be moved out of ingestIndexReferenceTask.py and into convertReferenceCatalog.py. We could potentially remove the base class entirely at that point, and just have ConvertReferenceCatalog by itself; the base class was just to let gen2/3 share most of the working code. See the note at the top of ingestIndexReferenceTask.py
Those cannot be moved until post-gen2, because the gen2 refcat config files have an explicit check that DatasetConfig live in ingestIndexReferenceTask.
In
DM-35671I did some relevant deprecations, including LoadIndexedReferenceObjectsConfig and LoadIndexedReferenceObjectsTask. I also made it so the (still poorly named; hence this ticket) preferred LoadReferenceObjectsConfig has a cal_ref_cat config field so that we can easily move from deprecated pexConfig.ConfigurableField(target=LoadIndexedReferenceObjectsTask) to undeprecated pexConfig.ConfigField(dtype=LoadReferenceObjectsConfig) (though we need to remove any overrides in configs of the deprecated cal_ref_cat field).Also note that currently there are tests test_loadReferenceObjects.py and test_referenceObjectLoader.py that both test different functionality of ReferenceObjectLoader. These should be consolidated on this ticket.