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

Upgrading display_ds9 to numpydoc status

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • display_ds9

    Description

      display_ds9 doc/status has already been done, so I will go through and upgrade to numpydoc status. 

      Attachments

        Issue Links

          Activity

            A few picky comments on the GitHub PR. Since I know you're on vacation, and since I'm about to spend ~10 hours sitting on a plane, I'll see if I can fix up some of these myself (but no promises) — watch this space.

            swinbank John Swinbank added a comment - A few picky comments on the GitHub PR. Since I know you're on vacation, and since I'm about to spend ~10 hours sitting on a plane, I'll see if I can fix up some of these myself (but no promises) — watch this space.
            swinbank John Swinbank added a comment - - edited

            Entertained myself on the flight producing https://github.com/lsst/display_ds9/pull/13 which I think addresses my comments.

            Still struggled a bit with naming “DS9” — I'm sure it shouldn't be `ds9` (it's not a cross-reference), but I considered ``ds9`` (is it a code literal?) and :command:`ds9` (which would refer to the executable itself) before settling on just DS9 (which is the name of the application, and I think is right in context.).

            swinbank John Swinbank added a comment - - edited Entertained myself on the flight producing https://github.com/lsst/display_ds9/pull/13 which I think addresses my comments. Still struggled a bit with naming “DS9” — I'm sure it shouldn't be `ds9` (it's not a cross-reference), but I considered ``ds9`` (is it a code literal?) and :command:`ds9` (which would refer to the executable itself) before settling on just DS9 (which is the name of the application, and I think is right in context.).

            Hey ebellm — just wanted to check this didn't get forgotten.

            swinbank John Swinbank added a comment - Hey ebellm — just wanted to check this didn't get forgotten.

            People

              ebellm Eric Bellm
              dstaker David Staker [X] (Inactive)
              John Swinbank
              David Staker [X] (Inactive), John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.