Paul Price added a comment - 01/Feb/18 3:43 PM I have improved the mechanics of constructing the full-focalplane images, which removes all of the chatter except for the final warning ( Unable to create camera image ), which I have removed by adding the datasets to obs_comCam. Robert Lupton , could you please verify this is what you had in mind? price@pap-laptop:~/LSST/afw (tickets/DM-13437 %=) $ git sub commit 7660d09479ae2b41f848c45f68061a490dce8558 (HEAD -> tickets/DM-13437, origin/tickets/DM-13437) Author: Paul Price <price@astro.princeton.edu> Date: Thu Feb 1 16:42:07 2018 -0500   makeImageFromCamera: catch exceptions from retrieving image This allows the 'imageSource' to throw an exception rather than returning a nonsense input.   python/lsst/afw/cameraGeom/utils.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)     price@pap-laptop:~/LSST/pipe_drivers (tickets/DM-13437=) $ git sub commit 8489765068486868910d0320ccbc5976b59ca2db Author: Paul Price <price@astro.princeton.edu> Date: Thu Feb 1 15:16:58 2018 -0500   constructCalibs: fix camera image construction The dict of calibs has been indexed by the "ccdName" (a tuple of CCD-specific dataId values), but our ImageSource doesn't know how to construct one of those from the detector it's given. Instead, index the calibs by the detector ID.   python/lsst/pipe/drivers/constructCalibs.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-)   commit 7c519fb91fa09d4d9a0806229cdd7d212cb9845d (HEAD -> tickets/DM-13437, origin/tickets/DM-13437) Author: Paul Price <price@astro.princeton.edu> Date: Thu Feb 1 17:19:36 2018 -0500   constructCalibs: use fake image for full-focal plane if unavailable We have been returning 1x1 images if there's no image, and that generates annoying warnings with cryptic messages. Instead, generate a full-size fake image.   python/lsst/pipe/drivers/constructCalibs.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)     price@pap-laptop:~/LSST/obs_comCam (tickets/DM-13437=) $ git sub commit edd5a182fd96fb6f1fb0db265d783adae21a61c3 (HEAD -> tickets/DM-13437, origin/tickets/DM-13437) Author: Paul Price <price@astro.princeton.edu> Date: Thu Feb 1 17:29:15 2018 -0500   mapper: add full focal-plane images of calibs Very useful for getting a quick feel of the quality of what you just built.   policy/comCamMapper.yaml | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+)

Paul Price added a comment - 01/Feb/18 3:45 PM GitHub: https://github.com/lsst/afw/pull/318 https://github.com/lsst/pipe_drivers/pull/51 https://github.com/lsst/obs_comCam/pull/9

Robert Lupton added a comment - 03/Feb/18 9:31 AM Minor comments on GitHub

Paul Price added a comment - 05/Feb/18 11:42 AM Addressed review comments and merged to master.