Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-19331

Fix detector name for HSC

    Details

      Description

      The HSC detector name was being read from the T_CCDSN header but this value does not match common usage. The names are defined in the obs_subaru camera geom files of the form RAFT_NUMBER where raft is 0 and 1 and numbers are zero-padded two digits.

      Currently raft is not a concept used in obs_subaru.

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          After discussing this on slack the proposal is to:

          • Define detector_group and detector_name in the LSST style.
          • Add new property of detector_unique_name which would combine the group and name.
          Show
          tjenness Tim Jenness added a comment - After discussing this on slack the proposal is to: Define detector_group and detector_name in the LSST style. Add new property of detector_unique_name which would combine the group and name.
          Hide
          tjenness Tim Jenness added a comment -

          This is the formal review request although the GitHub review has completed. I have also changed DECam to use their group of N and S. I probably should also change CFHT to use detector names of ccdX rather than the nick names.

          Show
          tjenness Tim Jenness added a comment - This is the formal review request although the GitHub review has completed. I have also changed DECam to use their group of N and S. I probably should also change CFHT to use detector names of ccdX rather than the nick names.
          Hide
          krughoff Simon Krughoff added a comment -

          Looks great.

          Show
          krughoff Simon Krughoff added a comment - Looks great.

            People

            • Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Simon Krughoff
              Watchers:
              Simon Krughoff, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: