Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_mosaic
-
Labels:
-
Story Points:1
-
Team:Data Release Production
Description
Furusawa-san pointed out that coaddDriver can produce outputs even if mosaicking has not been run though doApplyUberCal=True (which is the default value for HSC). This appears to be due to allowing null values of the wcs and ffp in updateExposure.py.
We should allow the functions in there to fail if the appropriate values are not available, or the user can be deceived as to what corrections have actually been applied.
Attachments
Issue Links
- blocks
-
DM-10350 Next hscPipe 5.0 beta release
- Done
Bob Armstrong, are you able to review this, please?
Lauren MacArthur, if you're able to have a look, that'd be great too, but I know you're away at the moment.
price@pap-laptop:~/LSST/meas_mosaic (tickets/DM-10546=) $ git sub
commit 6ea395a4ce466e7d67e8004b2d81def6a09ac6c2
Author: Paul Price <price@astro.princeton.edu>
Date: Wed May 17 15:58:11 2017 +0900
updateExposure: allow exceptions if data doesn't exist
The various functions to retrieve and apply the correction data
were silently proceeding in the event that the data did not exist,
misleading the user. Instead, allow exceptions to bubble up ---
the user asked us to retrieve and/or apply the correction, so if
we can't do that there's a problem that the user needs to know
about.
python/lsst/meas/mosaic/updateExposure.py | 101 ++++++++++++++----------------
1 file changed, 48 insertions(+), 53 deletions(-)