Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: pipe_analysis
-
Labels:None
-
Story Points:8
-
Epic Link:
-
Sprint:DRP S19-6b, DRP F19-1, DRP F19-2, DRP F19-4
-
Team:Data Release Production
Description
The qa scripts in pipe_analysis were written and tuned based entirely on HSC-SSP data. Now that we are moving to generalizing the scripts to work on any dataset produced by the lsst stack, certain hard-coded values need to be adapted to be appropriate for the dataset of interest. As Eli Rykoff points out in DM-18635, one such setting that requires adjustment is the minimum flux limit used for computing the basic statistics (mean and stddev) for the various metrics. While it is currently a config parameter (whose default was set for HSC and specified by RHL), it would be even better to set this limit based on a S/N cut (e.g. S/N>100 would also match what validate_drp is using). Once the new setting has been tested and shown to be the better option, make it the default, but allow for an override option in the configs to force a fixed mag cut (i.e the current behavior).
Attachments
Issue Links
- is blocked by
-
DM-19828 Investigate oddness in error propagation with meas_mosaic's photoCalib
- Done