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

Add notes about dataset contents ap_verify_ci_cosmos_pdr2

    XMLWordPrintable

    Details

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

      Description

      The ap_verify_ci_cosmos_pdr2 readme doesn't contain information about what is in the dataset (see ci_hits2015 for comparison). Having this makes it a lot easier to make queries when working in a run of the ci product.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment - - edited

          Probably to minimize duplication with https://github.com/lsst/ap_verify_ci_cosmos_pdr2/blob/main/doc/ap_verify_ci_cosmos_pdr2/index.rst. Given that doc/ will likely never be published, it's probably a good idea to duplicate or even merge.

          Show
          krzys Krzysztof Findeisen added a comment - - edited Probably to minimize duplication with https://github.com/lsst/ap_verify_ci_cosmos_pdr2/blob/main/doc/ap_verify_ci_cosmos_pdr2/index.rst . Given that doc/ will likely never be published, it's probably a good idea to duplicate or even merge.
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Meredith Rawls: trivial review for the readme update. I probably could have just done it on a user branch, since there wasn't much to add.

          https://github.com/lsst/ap_verify_ci_cosmos_pdr2/pull/30

          Show
          Parejkoj John Parejko added a comment - Meredith Rawls : trivial review for the readme update. I probably could have just done it on a user branch, since there wasn't much to add. https://github.com/lsst/ap_verify_ci_cosmos_pdr2/pull/30
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Krzysztof Findeisen: thanks for that link: I've added some of that to the readme. I don't know if we should delete that doc/ stuff or not, since as you say, it won't ever be published.

          Show
          Parejkoj John Parejko added a comment - Krzysztof Findeisen : thanks for that link: I've added some of that to the readme. I don't know if we should delete that doc/ stuff or not, since as you say, it won't ever be published.
          Hide
          Permalink
          mrawls Meredith Rawls added a comment -

          Thanks for adding this clarifying info!

          Show
          mrawls Meredith Rawls added a comment - Thanks for adding this clarifying info!

            People

            Assignee:
            Parejkoj John Parejko
            Reporter:
            Parejkoj John Parejko
            Reviewers:
            Meredith Rawls
            Watchers:
            Brianna Smart, Eric Bellm, Erin Howard, Ian Sullivan, John Parejko, Krzysztof Findeisen, Meredith Rawls, Neven Caplar
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.