Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-617

Update HSC's focal plane camera geometry to use units of millimeters

    XMLWordPrintable

    Details

      Description

      The obs_subaru camera geometry currently uses an effective focal plane scaling of "unity" (i.e. plateScale = 1 arcsec/mm, pixelSize = 1 mm). All other LSST camera geometries are defined in actual physical units: millimeters on the focal plane. For the work being done in DM-19943 (and see RFC-616), we need to have the correct CCD plate scale in the camera geometry to properly reflect the pixel-to-focal plane conversion.

      In addition to this unit conversion being a good idea in its own right, the investigations in DM-19943 rendered us fairly confident that there is an inconsistency in the current HSC cameraGeom itself that was actually responsible for most of the problems. Updating the units thus has the added bonus of eliminating the inconsistency without us having to debug it further. Furthermore, HSC's non-conventional unit system has caused mild headaches for a few developers (e.g. Eli Rykoff's calibration work on fgcmcal), so the switch will benefit them as well.

      There are a few package affected and will require accommodation. These have already been exposed and tested on DM-20289 for singleFrame and skyCorrection and, in the context of ci_hsc, through coaddition and multiband with the cursory tests there (but without running jointcal, which is not yet included in any of our CI scripts). A more rigorous battle testing is being performed on DM-20548 (the ticket triggered by this RFC) which will include a full run of at least one tract from the RC2 dataset (i.e. running through singleFrameDriver, skyCorrection, jointcal, coaddDriver, and multibandDriver) to make sure the results are consistent with that of the latest weekly reprocessing run.

      One item of note: this change will effectively break meas_mosaic. While updating it to accommodate the unit update is not completely out of the question, it would take significant effort which does not seem worthwhile given the adoption of jointcal as our “uber calibration” algorithm. As such, this RFC is also effectively proposing the official retirement of meas_mosaic.

        Attachments

          Issue Links

            Activity

            lauren Lauren MacArthur created issue -
            lauren Lauren MacArthur made changes -
            Field Original Value New Value
            Link This issue is triggering DM-20548 [ DM-20548 ]
            swinbank John Swinbank made changes -
            Status Proposed [ 10805 ] Flagged [ 10606 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 20982 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 21035 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 21152 ]
            swinbank John Swinbank made changes -
            Status Flagged [ 10606 ] Board Recommended [ 11405 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 21252 ]
            lauren Lauren MacArthur made changes -
            Status Board Recommended [ 11405 ] Adopted [ 10806 ]
            lauren Lauren MacArthur made changes -
            Resolution Done [ 10000 ]
            Status Adopted [ 10806 ] Implemented [ 11105 ]
            lauren Lauren MacArthur made changes -
            Link This issue is triggering DM-21053 [ DM-21053 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 21423 ]

              People

              Assignee:
              lauren Lauren MacArthur
              Reporter:
              lauren Lauren MacArthur
              Watchers:
              Chris Morrison, Eli Rykoff, Gabriele Comoretto [X] (Inactive), Jim Bosch, John Parejko, John Swinbank, Joshua Meyers, Lauren MacArthur, Paul Price, Robert Lupton, Tim Jenness, Wil O'Mullane, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              13 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.