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

Use repositories more idiomatically

    Details

    • Story Points:
      3
    • Epic Link:
    • Sprint:
      AP S18-3
    • Team:
      Alert Production

      Description

      For forward-compatibility with DM-13163, ap_verify should create separate repositories for ingestion and calibration. In effect, the current "output repository" should be a convenient "workspace" directory but not a repository.

      In addition, the interface module pipeline_driver should make a distinction between input and output repositories, choosing the location of the latter instead of deferring the choice to ap_pipe. Neither CmdLineTasks, nor in the future Pipelines, are responsible for output paths.

      Because the API will change again as part of DM-13163, changes to the interface between ap_pipe and ap_verify should be kept minimal; most likely, this will entail removing the ill-advised wrappers added in DM-12257 and calling the existing functions from pipeline_driver again.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            Hi John Swinbank, can you review this ticket? I make some changes to ap_verify's command-line interface, but they're the minimal ones required to avoid the previous "repository of repositories" behavior.

            Show
            krzys Krzysztof Findeisen added a comment - - edited Hi John Swinbank , can you review this ticket? I make some changes to ap_verify 's command-line interface, but they're the minimal ones required to avoid the previous "repository of repositories" behavior.
            Hide
            swinbank John Swinbank added a comment -

            Sorry for being slow to review.

            Changes look good: thank you!

            I think a few words of clarification in the documentation might help. See comments on the PR.

            Show
            swinbank John Swinbank added a comment - Sorry for being slow to review. Changes look good: thank you! I think a few words of clarification in the documentation might help. See comments on the PR.

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                krzys Krzysztof Findeisen
                Reviewers:
                John Swinbank
                Watchers:
                John Swinbank, Krzysztof Findeisen, Meredith Rawls
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel