New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-5877: Use Afterburners to clean up aperture correction logic #42
Conversation
|
||
|
||
class ABClassificationConfig(AfterburnerPluginConfig): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the blank line since it doesn't add anything and adds potential for the parts being separated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check
c1e5048
to
0577a7c
Compare
copyColumns = lsst.pex.config.DictField( | ||
keytype=str, itemtype=str, doc="Mapping of reference columns to source columns", | ||
default={"id": "objectId", "parent": "parentObjectId", "deblend_nChild": "deblend_nChild"} | ||
) | ||
|
||
def setDefaults(self): | ||
# Make afterburners a NoOp by default as not modelFlux is setup by default in | ||
# ForcedMeasurementTask |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"not" - > "no", "NoOp" -> "no-op" (don't want CamelCase to imply it's a class).
The afterburner task contained no reference to the plugin map for afterburner plugins. This make it impossible to reference the plugins though the class itself. Added map to facilitate this functionality
The classification plugin is now a afterburner plugin instead of a measurement plugin. Created a new classificatioin module to hold all future classification plugins. Removed classification run order, as it is now the same run ordering as afterburners
Remove all the logic of aperture corrections from the measurement plugins. In the future MeasureApCorr and ApplyApCorr tasks should be called directly when that functionality is required.
Since aperture corrections have been moved out of the base measurement task, application of aperture corrections and the afterburner task must be added to tasks which need these functionalities. Since the forced measurement task has no default model flux, afterburners are set to a NoOp by default.
No description provided.