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

Remove ap_verify_hits2015 dataset

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • ap_verify
    • None

    Description

      Retire the ap_verify_hits2015 dataset using the standard package removal process, leaving it as a legacy package in lsst-dm.

      Attachments

        Issue Links

          Activity

            krzys Krzysztof Findeisen added a comment - - edited

            The ap_verify documentation (at minimum, Running ap_verify from the command line) has warnings to use caution when running entire datasets. Since we do not intend to create large datasets in the future, we should remove any such warnings (on all documentation pages) along with the ap_verify_hits2015 dataset.

            krzys Krzysztof Findeisen added a comment - - edited The ap_verify documentation (at minimum, Running ap_verify from the command line ) has warnings to use caution when running entire datasets. Since we do not intend to create large datasets in the future, we should remove any such warnings (on all documentation pages) along with the ap_verify_hits2015 dataset.
            krzys Krzysztof Findeisen added a comment - - edited

            See https://github.com/search?q=org%3Alsst+ap_verify_hits2015&type=code for a list of existing cross-references:

            • lsst/ap_pipe:doc/lsst.ap.pipe/getting-started.rst
            • Multiple TeX files in lsst/LDM-533:cases Archival use only, no changes needed.
            krzys Krzysztof Findeisen added a comment - - edited See https://github.com/search?q=org%3Alsst+ap_verify_hits2015&type=code for a list of existing cross-references: lsst/ap_pipe:doc/lsst.ap.pipe/getting-started.rst Multiple TeX files in lsst/LDM-533:cases Archival use only, no changes needed.
            krzys Krzysztof Findeisen added a comment - - edited

            Thanks for agreeing to review this, harshit! In addition to moving ap_verify_hits2015 to https://github.com/lsst-dm/legacy-ap_verify_hits2015 (which is not really reviewable), there are four PRs: ap_pipe#145 (not visible in Jira sidebar), ap_verify#193, repos#145 (policy doesn't require a review, but I'm listing it for completeness), and lsstsw#313.

            krzys Krzysztof Findeisen added a comment - - edited Thanks for agreeing to review this, harshit ! In addition to moving ap_verify_hits2015 to https://github.com/lsst-dm/legacy-ap_verify_hits2015 (which is not really reviewable), there are four PRs: ap_pipe#145 (not visible in Jira sidebar), ap_verify#193 , repos#145 (policy doesn't require a review, but I'm listing it for completeness), and lsstsw#313 .

            People

              krzys Krzysztof Findeisen
              krzys Krzysztof Findeisen
              Harshit Rai
              Eric Bellm, Harshit Rai, Ian Sullivan, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.