Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_algorithms
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:Alert Production F16 - 6
-
Team:Alert Production
Description
Restore the registry for star selectors that was lost in DM-5532, now that tasks in registries can be used as subtasks.
Also use the registry where appropriate.
I updated MeasurePsfTask and MeasureApCorrTask to specify the star selector using the registry. This required minor changes to a few config override files and unit tests.
This consists of small changes to a number of packages. Some packages contain star selectors that need to register themselves, and some packages contain config overrides that needed updating.
One possibly controversial change I made was to rename the base class for star selectors from StarSelectorTask to BaseStarSelectorTask, in order to clarify that it is an abstract base class.This change is fairly innocuous in that it only affects star selectors (since they inherit from the base class), not users of those star selectors. I used a similar naming convention in
DM-6077making PSF determiners into tasks, which is what prompted the change.