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

obs_decam still has ccdnum baked in

    Details

      Description

      Various butler requests against obs_decam fail unless ccdnum is included in the dataId. Although DM-6585 took care of some of the ccdnum use in obs_decam, it is still baked-in to various parts of decamMapper.py, among others. Some examples:

      >>> self.butler.get("linearizer", ccd=50)
      *** KeyError: 'ccdnum'
       
      >>> butler.get("ccdExposureId", visit=229388)
      *** KeyError: 'ccdnum'
       
      >>> butler.get("defects", ccd=11)
      *** sqlite3.OperationalError: no such column: ccdnum
      

        Attachments

          Issue Links

            Activity

            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            I don't think this is a bug. DM-6585 was meant to provide a (temporary, I'd agree) way to use ccd for raw/calexp/src data, but not other dataset types.

            If we can all agree, replacing all ccdnum with ccd could be straightforward (i.e. no longer support using ccdnum). RFC will be needed I think.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - I don't think this is a bug. DM-6585 was meant to provide a (temporary, I'd agree) way to use ccd for raw/calexp/src data, but not other dataset types. If we can all agree, replacing all ccdnum with ccd could be straightforward (i.e. no longer support using ccdnum ). RFC will be needed I think.
            Hide
            ctslater Colin Slater added a comment -

            I agree with Hsin-Fang Chiang and was just making the same point to John in slack, and I think there will be a more general solution (in terms of a specification on obs packages) proposed in the near future.

            Show
            ctslater Colin Slater added a comment - I agree with Hsin-Fang Chiang and was just making the same point to John in slack, and I think there will be a more general solution (in terms of a specification on obs packages) proposed in the near future.
            Hide
            tjenness Tim Jenness added a comment -

            In butler gen 3 it's called "detector".

            Show
            tjenness Tim Jenness added a comment - In butler gen 3 it's called "detector".
            Hide
            Parejkoj John Parejko added a comment -

            Given that gen3 handles this the same everywhere, I'm "won't fixing" it.

            Show
            Parejkoj John Parejko added a comment - Given that gen3 handles this the same everywhere, I'm "won't fixing" it.

              People

              • Assignee:
                Unassigned
                Reporter:
                Parejkoj John Parejko
                Watchers:
                Colin Slater, Hsin-Fang Chiang, John Parejko, Kian-Tat Lim, Nate Pease, Simon Krughoff, Tim Jenness, Yusra AlSayyad
              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel