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

Accept idiomatic input repositories

    Details

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

      Description

      Currently, ap_pipe requires a single input repository with two directories, ingested and calibingested, each a repository in its own right. This will be a problem for DM-13163 because most command-line tasks allow the URIs of the data and calib repositories to be independent, and in any case the Stack convention is different from our current usage.

      This ticket will change the command line to accept a separate calib repository (the argument should behave the same as processCcd.py --calib?). The API to ap_verify will not be changed, as it will almost certainly change when DM-13163 is implemented.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Hi Meredith Rawls, can you review this small ticket? It changes the command-line interface to accept more generic repos but doesn't do anything about ap_verify making nonstandard ones.

            Show
            krzys Krzysztof Findeisen added a comment - Hi Meredith Rawls , can you review this small ticket? It changes the command-line interface to accept more generic repos but doesn't do anything about ap_verify making nonstandard ones.
            Hide
            mrawls Meredith Rawls added a comment -

            This small change is fine, provided you are able to successfully run the code with a variety of inputs as intended. This reminded me that there are essentially no tests for ap_pipe, which is largely my fault. I'll make a ticket for that.

            Show
            mrawls Meredith Rawls added a comment - This small change is fine, provided you are able to successfully run the code with a variety of inputs as intended. This reminded me that there are essentially no tests for ap_pipe , which is largely my fault. I'll make a ticket for that.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: