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

Get rid of ProcessCcdSdssTask and ProcessCcdDecamTask

    Details

      Description

      Update ProcessCcdTask so that it can be used with different datasete types as appropriate for the ISR task. This will allow us to get rid of obs-specific variants ProcessCcdSdssTask and ProcessCcdDecamTask

      The plan is to change ProcessCcdTask as follows:

      • set doMakeDataRefList=False in the call to add_id_argument
      • get the dataset type from the ISR task (default to "raw") and set it in data container
      • make the dataRef list by calling makeDataRefList on the data container

      Question for DECam folks: do you want two executable scripts for DECam (one that processes data from the community pipeline and one that performs ISR)? Or do you prefer one exectutable (in which case you switch between performing ISR and reading the output of the community pipeline output by retargeting the ISR task)? If you prefer one binary, then which should be the default: perform ISR or read the output of the community pipeline?

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            It turns out John Parejko can squeeze it in, so I've assigned it to him. I know Nate Lust is very busy right now.

            Show
            rowen Russell Owen added a comment - It turns out John Parejko can squeeze it in, so I've assigned it to him. I know Nate Lust is very busy right now.
            Hide
            Parejkoj John Parejko added a comment -

            Mostly documentation cleanups. Otherwise, good to go.

            Show
            Parejkoj John Parejko added a comment - Mostly documentation cleanups. Otherwise, good to go.
            Hide
            rowen Russell Owen added a comment -

            Merged to master

            Show
            rowen Russell Owen added a comment - Merged to master
            Hide
            rowen Russell Owen added a comment -

            Please review the changes in validate_drp.

            Show
            rowen Russell Owen added a comment - Please review the changes in validate_drp.
            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            Changes to validate_drp look good.
            Ready to merge.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Changes to validate_drp look good. Ready to merge.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Michael Wood-Vasey
                Watchers:
                David Nidever [X] (Inactive), Hsin-Fang Chiang, Jim Bosch, John Parejko, John Swinbank, Lauren MacArthur, Michael Wood-Vasey, Paul Price, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel