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

Remove ap_association subtasks from ap_pipe. Change to single DiaPipelineTask subtask.

    Details

      Description

      Now that the new Gen3 subtask is created, remove the previously directly used ap_association subtasks from ap_pipe and replace with a DiaPipelineTask initialized as a subtask.

        Attachments

          Issue Links

            Activity

            Show
            cmorrison Chris Morrison added a comment - - edited Jenkins:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30939/pipeline/
            Hide
            cmorrison Chris Morrison added a comment -

            Should be quick hopefully. I'll be on vacation till the 2nd (after the 23rd ends) so don't worry about rushing to finish this ticket.

            I have tested this with the CI run in ap_verify and confirmed that the outputs are reasonable.

            Show
            cmorrison Chris Morrison added a comment - Should be quick hopefully. I'll be on vacation till the 2nd (after the 23rd ends) so don't worry about rushing to finish this ticket. I have tested this with the CI run in ap_verify and confirmed that the outputs are reasonable.
            Hide
            krzys Krzysztof Findeisen added a comment -

            I found a couple of bugs in the new ap_pipe code; otherwise looks good.

            Show
            krzys Krzysztof Findeisen added a comment - I found a couple of bugs in the new ap_pipe code; otherwise looks good.
            Show
            cmorrison Chris Morrison added a comment - Jenkins after review:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30944/pipeline

              People

              • Assignee:
                cmorrison Chris Morrison
                Reporter:
                cmorrison Chris Morrison
                Reviewers:
                Krzysztof Findeisen
                Watchers:
                Chris Morrison, Eric Bellm, John Swinbank, Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: