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

convert ip_diffim documentation to numpydoc

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
    • Story Points:
      4
    • Sprint:
      AP F18-4, AP F18-5, AP F18-6, AP S19-1, AP S19-2
    • Team:
      Alert Production

      Attachments

        Issue Links

          Activity

          No builds found.
          dstaker David Staker [X] (Inactive) created issue -
          dstaker David Staker [X] (Inactive) made changes -
          Field Original Value New Value
          Risk Score 0
          swinbank John Swinbank made changes -
          Epic Link DM-14447 [ 80385 ]
          swinbank John Swinbank made changes -
          Sprint AP F18-4 [ 749 ]
          Team Alert Production [ 10300 ]
          swinbank John Swinbank made changes -
          Sprint AP F18-4 [ 749 ] AP F18-4, AP F18-5 [ 749, 750 ]
          swinbank John Swinbank made changes -
          Sprint AP F18-4, AP F18-5 [ 749, 750 ] AP F18-4, AP F18-5, AP F18-6 [ 749, 750, 751 ]
          swinbank John Swinbank made changes -
          Assignee David Staker [ dstaker ] John Swinbank [ swinbank ]
          swinbank John Swinbank made changes -
          Epic Link DM-14447 [ 80385 ] DM-16722 [ 235355 ]
          swinbank John Swinbank made changes -
          Sprint AP F18-4, AP F18-5, AP F18-6 [ 749, 750, 751 ] AP F18-4, AP F18-5, AP F18-6, AP S19-1 [ 749, 750, 751, 825 ]
          sullivan Ian Sullivan made changes -
          Assignee John Swinbank [ swinbank ] Ian Sullivan [ sullivan ]
          sullivan Ian Sullivan made changes -
          Labels numpydoc
          sullivan Ian Sullivan made changes -
          Component/s ip_diffim [ 10743 ]
          sullivan Ian Sullivan made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          swinbank John Swinbank made changes -
          Sprint AP F18-4, AP F18-5, AP F18-6, AP S19-1 [ 749, 750, 751, 825 ] AP F18-4, AP F18-5, AP F18-6, AP S19-1, AP S19-2 [ 749, 750, 751, 825, 830 ]
          sullivan Ian Sullivan made changes -
          Link This issue relates to DM-17458 [ DM-17458 ]
          Hide
          sullivan Ian Sullivan added a comment -

          Thanks for being willing to take a look at this, there are a lot of lines of code changed. I tried to go through everything and add docstrings to every function, though I ran out of time at the end and left filling in a few of the details for a future ticket (DM-17458). I tried to leave a TODO with that ticket number instead of just leaving it blank, but please let me know if something else would be better filler.

          Show
          sullivan Ian Sullivan added a comment - Thanks for being willing to take a look at this, there are a lot of lines of code changed. I tried to go through everything and add docstrings to every function, though I ran out of time at the end and left filling in a few of the details for a future ticket ( DM-17458 ). I tried to leave a TODO with that ticket number instead of just leaving it blank, but please let me know if something else would be better filler.
          sullivan Ian Sullivan made changes -
          Reviewers Jonathan Sick [ jsick ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          sullivan Ian Sullivan made changes -
          Story Points 4
          Hide
          jsick Jonathan Sick added a comment -

          Thanks for doing this! I had to time box the review because obviously there's a lot to cover, and a number of things can be refined with further iterative development. I hope that the comments I have provided are at least educational and can frame future work.

          In general, an easy win is to ensure that all the descriptions are sentences (capitalized and ending with a period).

          I also noticed that there are references to DipoleFitAlgorithm, but that class isn't part of the public API documentation. You might want to look into either making that API public in `lsst.ip.diffim` or adding another `automodapi` directive that points to the module where DipoleFitAlgorithm is located (choose whichever is appropriate).

          Show
          jsick Jonathan Sick added a comment - Thanks for doing this! I had to time box the review because obviously there's a lot to cover, and a number of things can be refined with further iterative development. I hope that the comments I have provided are at least educational and can frame future work. In general, an easy win is to ensure that all the descriptions are sentences (capitalized and ending with a period). I also noticed that there are references to DipoleFitAlgorithm, but that class isn't part of the public API documentation. You might want to look into either making that API public in `lsst.ip.diffim` or adding another `automodapi` directive that points to the module where DipoleFitAlgorithm is located (choose whichever is appropriate).
          jsick Jonathan Sick made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Hide
          sullivan Ian Sullivan added a comment -

          Further improvements to the documentation are left for DM-17458.

          Show
          sullivan Ian Sullivan added a comment - Further improvements to the documentation are left for DM-17458 .
          sullivan Ian Sullivan made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          bsmart Brianna Smart made changes -
          Link This issue relates to DM-37801 [ DM-37801 ]

            People

            Assignee:
            sullivan Ian Sullivan
            Reporter:
            dstaker David Staker [X] (Inactive)
            Reviewers:
            Jonathan Sick
            Watchers:
            David Staker [X] (Inactive), Ian Sullivan, Jonathan Sick
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.