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

Convert PipelineTasks to new api

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr, pipe_tasks
    • Labels:
      None

      Description

      DM-20205 introduces a new api for pipeline tasks. This ticket will update the existing pipeline tasks to use this new api.

        Attachments

          Issue Links

            Activity

            Hide
            nlust Nate Lust added a comment -

            A stack with both this ticket and 20205 are built on lsst-dev at /home/nate2/lsstsw and tagged as current

            Show
            nlust Nate Lust added a comment - A stack with both this ticket and 20205 are built on lsst-dev at /home/nate2/lsstsw and tagged as current
            Hide
            jbosch Jim Bosch added a comment -

            Review complete; comments on a few PRs.  I suspect Yusra AlSayyad is probably the person best suited to reply to many of my comments in pipe_tasks.

            Nate Lust, with what time you have remaining before you leave please focus on replying on PRs rather than making changes; I can make changes later but would prefer not to do so unilaterally.

            I also think we should change the injected PipelineTaskConfig.connections.connectionClass to ConnectionsClass for similarity with ConfigClass (commented to that effect on the pipe_base PR for DM-20205.

            Show
            jbosch Jim Bosch added a comment - Review complete; comments on a few PRs.  I suspect Yusra AlSayyad is probably the person best suited to reply to many of my comments in pipe_tasks. Nate Lust , with what time you have remaining before you leave please focus on replying on PRs rather than making changes; I can make changes later but would prefer not to do so unilaterally. I also think we should change the injected PipelineTaskConfig.connections.connectionClass to ConnectionsClass for similarity with ConfigClass (commented to that effect on the pipe_base PR for DM-20205 .
            Hide
            krzys Krzysztof Findeisen added a comment -

            I just pushed a documentation fix to the verify branch that slipped through the cracks during PCW. Sorry for the inconvenience!

            Show
            krzys Krzysztof Findeisen added a comment - I just pushed a documentation fix to the verify branch that slipped through the cracks during PCW. Sorry for the inconvenience!

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              nlust Nate Lust
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, Krzysztof Findeisen, Nate Lust
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: