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-42160: eo_pipe/cp_verify parity: dark #226
Conversation
@@ -612,6 +615,33 @@ def setFilter(exp, filterLabel): | |||
""" | |||
pass | |||
|
|||
def calibStats(self, exp, calibrationType): |
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.
This could be a static method, I guess.
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.
Left alone, as this now uses a config parameter.
metadata.set(f"LSST CALIB {calibrationType.upper()} {amp.getName()} DISTRIBUTION 84-PCT", p84) | ||
metadata.set(f"LSST CALIB {calibrationType.upper()} {amp.getName()} DISTRIBUTION 95-PCT", p95) | ||
metadata.set(f"LSST CALIB {calibrationType.upper()} {amp.getName()} DISTRIBUTION 100-PCT", p100) | ||
|
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.
Instead of hard-coding these, could we take the percentile values to compute as an input, and loop over those?
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.
You're right, hard coding this is likely to be annoying in the future. I've moved the percentile levels to a config option, and have this method zip
the levels and measured values into a second loop. Then, if someone decides the quartiles are useful too, it's just a config change.
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.
PS metadata[key] = value
should work fine.
cb5a7d5
to
267548e
Compare
Add calibration distribution statistics.