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

Upgrading display_ds9 to numpydoc status

    XMLWordPrintable

    Details

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

      Description

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

        Attachments

          Issue Links

            Activity

            Hide
            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.

            Show
            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.
            Hide
            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.).

            Show
            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.).
            Hide
            swinbank John Swinbank added a comment -

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

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

              People

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

                Dates

                Created:
                Updated:
                Resolved:
                Start date:

                  CI Builds

                  No builds found.