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-25990: Reprocess HSC COSMOS medium dataset with ap_pipe #419
Conversation
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.
Looks good, but please see my comments on the error and log messages.
raise ValueError("If you are scaling the diffim variance, it is pointless to also " | ||
"scale the template variance. Please choose one or the other.") |
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.
I would avoid error messages that judge the user. How about "Scaling the difference image variance and template variance are both set. Please choose one or the other."
@@ -601,6 +608,7 @@ def run(self, exposure=None, selectSources=None, templateExposure=None, template | |||
|
|||
if self.config.doSubtract: | |||
if self.config.doScaleTemplateVariance: | |||
self.log.info("Rescaling template variance") | |||
templateVarFactor = self.scaleVariance.run( | |||
templateExposure.getMaskedImage()) | |||
self.metadata.add("scaleTemplateVarianceFactor", templateVarFactor) |
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.
Please add a log message stating the template variance scaling that is being used.
self.log.info("Rescaling diffim variance") | ||
diffimVarFactor = self.scaleVariance.run(subtractedExposure.getMaskedImage()) | ||
self.metadata.add("scaleDiffimVarianceFactor", diffimVarFactor) | ||
|
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.
Please add a log message stating the difference image variance scaling that is being used.
b139e35
to
2bbea52
Compare
2bbea52
to
5325971
Compare
This change adds a doScaleDiffimVariance config option, which was set True for this ticket's ap_pipe rerun.