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

Clarify how to work with ci_hsc's astrometry_net_data

    Details

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

      Description

      ci_hsc's README.rst contains a note on declaring & setting up the included reference catalogue data.

      I believe this was rendered obsolete by DM-5135, which automatically sets up the reference catalogue when ci_hsc itself is set up. Attempting to follow the documentation therefore produces confusing warning messages, and may break things.

      Please check if my understanding is correct and, if so, fix the documentation.

        Attachments

          Activity

          Hide
          wmwood-vasey Michael Wood-Vasey added a comment -

          John Swinbank I agree. Thank you for catching this.

          Show
          wmwood-vasey Michael Wood-Vasey added a comment - John Swinbank I agree. Thank you for catching this.
          Hide
          wmwood-vasey Michael Wood-Vasey added a comment -

          Text updated:

          diff --git a/README.rst b/README.rst
          index becd86b..d8eeef1 100644
          --- a/README.rst
          +++ b/README.rst
          @@ -30,13 +30,8 @@ Reference catalog
           -----------------
           
           An appropriate astrometric reference catalog is provided in the
          -``sdss-dr9-fink-v5b`` directory. It must be declared to `EUPS`_ and set up
          -before using this package::
          -
          -  $ eups declare -r ${CI_HSC_DIR}/sdss-dr9-fink-v5b astrometry_net_data sdss-ci_hsc
          -  $ setup -j astrometry_net_data sdss-ci_hsc
          -
          -.. _EUPS: https://github.com/RobertLuptonTheGood/eups
          +``sdss-dr9-fink-v5b`` directory.  This is specified in `ups/ci_hsc.table`
          +and is set up automatically when ``ci_hsc`` is ``setup`` above.
          
          

          Show
          wmwood-vasey Michael Wood-Vasey added a comment - Text updated: diff --git a/README.rst b/README.rst index becd86b..d8eeef1 100644 --- a/README.rst +++ b/README.rst @@ -30,13 +30,8 @@ Reference catalog -----------------   An appropriate astrometric reference catalog is provided in the -``sdss-dr9-fink-v5b`` directory. It must be declared to `EUPS`_ and set up -before using this package:: - - $ eups declare -r ${CI_HSC_DIR}/sdss-dr9-fink-v5b astrometry_net_data sdss-ci_hsc - $ setup -j astrometry_net_data sdss-ci_hsc - -.. _EUPS: https://github.com/RobertLuptonTheGood/eups +``sdss-dr9-fink-v5b`` directory. This is specified in `ups/ci_hsc.table` +and is set up automatically when ``ci_hsc`` is ``setup`` above.
          Hide
          swinbank John Swinbank added a comment -

          Great, thank you!

          Show
          swinbank John Swinbank added a comment - Great, thank you!
          Hide
          wmwood-vasey Michael Wood-Vasey added a comment -

          Merged to master.

          Show
          wmwood-vasey Michael Wood-Vasey added a comment - Merged to master.

            People

            • Assignee:
              wmwood-vasey Michael Wood-Vasey
              Reporter:
              swinbank John Swinbank
              Reviewers:
              John Swinbank
              Watchers:
              John Swinbank, Michael Wood-Vasey
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel