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

Incorrect instructions in ap_verify readme

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None

      Description

      The ap_verify readme says to run python/lsst/ap/verify/ap_verify.py, even though this file has not been executable for almost a year. While the Sphinx documentation has been kept up-to-date, it appears the readme has bitrotted.

      Correct the example command line, and proofread the readme for any other out-of-date or missing information.

        Attachments

          Activity

          Hide
          krzys Krzysztof Findeisen added a comment -

          Also make sure all examples in both the readme and Sphinx docs use real data IDs.

          Show
          krzys Krzysztof Findeisen added a comment - Also make sure all examples in both the readme and Sphinx docs use real data IDs.
          Hide
          ebellm Eric Bellm added a comment -

          Per our discussion in sprint planning, we suggested that the README should instead direct users to the sphinx docs to minimize duplicated information.

          Show
          ebellm Eric Bellm added a comment - Per our discussion in sprint planning, we suggested that the README should instead direct users to the sphinx docs to minimize duplicated information.
          Hide
          krzys Krzysztof Findeisen added a comment -

          Eric Bellm, can you please review this?

          Show
          krzys Krzysztof Findeisen added a comment - Eric Bellm , can you please review this?
          Hide
          ebellm Eric Bellm added a comment -

          looks good.

          Show
          ebellm Eric Bellm added a comment - looks good.

            People

            • Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Eric Bellm
              Watchers:
              Eric Bellm, Krzysztof Findeisen, Simon Krughoff
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel