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

Lightly restructure ap_association tasks and create new DiaObject/DiaSource pre-loading task.

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_association, ap_pipe
    • Labels:
      None

      Description

      As part of DM-21886 we can better future proof subtasks within ap_association by creating a new task to load from the Apdb and lightly restructure other association tasks to take preloaded data instead of loading from the Apdb as needed.

        Attachments

          Issue Links

            Activity

            Hide
            cmorrison Chris Morrison added a comment - - edited

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30896/pipeline/

            Bulk of code change is the new task and unittest and also modifications to the AssociationTask unittests to accommodate the "pre-loading" new workflow.

            Show
            cmorrison Chris Morrison added a comment - - edited Jenkins:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30896/pipeline/ Bulk of code change is the new task and unittest and also modifications to the AssociationTask unittests to accommodate the "pre-loading" new workflow.
            Hide
            krzys Krzysztof Findeisen added a comment -

            I have some questions about the ap_association design, but overall things look good.

            Show
            krzys Krzysztof Findeisen added a comment - I have some questions about the ap_association design, but overall things look 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/30904/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:

                  Summary Panel