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

Convert DCR templates to PipelineTasks

    Details

    • Story Points:
      8
    • Sprint:
      AP F19-6 (November), AP S20-1 (December), AP S20-2 (January), AP S20-3 (February)
    • Team:
      Alert Production

      Description

      The DCR template generation code inherits from AssembleCoaddTask which has already been converted to a PipelineTask, so it should hopefully not be too much extra work to update DcrAssembleCoaddTask. I expect the majority of the work will involve adding a runQuantum method and a DcrAssembleCoaddConnections class.

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            Updating the story points to reflect the effort taken to get DcrAssembleCoaddTask running as a pipelineTask, and my improved understanding of the remaining work needed to fully convert the task.

            Show
            sullivan Ian Sullivan added a comment - Updating the story points to reflect the effort taken to get DcrAssembleCoaddTask running as a pipelineTask, and my improved understanding of the remaining work needed to fully convert the task.
            Hide
            sullivan Ian Sullivan added a comment -

            These changes can only be merged after DM-23008 and DM-22648

            This has been verified to work using HSC r-band test data. There should be no DCR in that dataset, but I was able to verify that the Gen 3 butler and pipelinetask functionality works.

            Show
            sullivan Ian Sullivan added a comment - These changes can only be merged after DM-23008 and DM-22648 This has been verified to work using HSC r-band test data. There should be no DCR in that dataset, but I was able to verify that the Gen 3 butler and pipelinetask functionality works.
            Hide
            sullivan Ian Sullivan added a comment -

            The pull request might not show up automatically, but can be found here:
            https://github.com/lsst/pipe_tasks/pull/339

            Show
            sullivan Ian Sullivan added a comment - The pull request might not show up automatically, but can be found here: https://github.com/lsst/pipe_tasks/pull/339
            Hide
            nlust Nate Lust added a comment -

            There are things to address, but it looks nearly there.

            Show
            nlust Nate Lust added a comment - There are things to address, but it looks nearly there.
            Hide
            nlust Nate Lust added a comment -

            Im sorry, I thought I had pressed reviewed on this a while ago, but just noticed it still in my list.

            Show
            nlust Nate Lust added a comment - Im sorry, I thought I had pressed reviewed on this a while ago, but just noticed it still in my list.
            Hide
            sullivan Ian Sullivan added a comment -

            Not to worry, I saw the review on GitHib. I've been sick for a few days so I haven't been able to wrap up my changes quite yet.

            Show
            sullivan Ian Sullivan added a comment - Not to worry, I saw the review on GitHib. I've been sick for a few days so I haven't been able to wrap up my changes quite yet.

              People

              • Assignee:
                sullivan Ian Sullivan
                Reporter:
                sullivan Ian Sullivan
                Reviewers:
                Nate Lust
                Watchers:
                Eric Bellm, Ian Sullivan, Jim Bosch, John Swinbank, Nate Lust
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: