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

Convert assembleCoadd.py to numpydoc

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      AP S18-6
    • Team:
      Alert Production

      Description

      Finish porting pipe_tasks docs from doxygen to numpydocs following Jonathan Sick's hack day instructions.

        Attachments

          Issue Links

            Activity

            No builds found.
            mrawls Meredith Rawls created issue -
            mrawls Meredith Rawls made changes -
            Field Original Value New Value
            Rank Ranked higher
            mrawls Meredith Rawls made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S18-5 [ 685 ] AP S18-6 [ 686 ]
            krzys Krzysztof Findeisen made changes -
            Risk Score 0
            Hide
            mrawls Meredith Rawls added a comment -

            Jonathan Sick, would you please review this? Note that the scope changed significantly as I realized how much work it was to manually convert all the docstrings in a python module. This ticket now includes numpydoc conversions for only assembleCoadd.py. 

            Show
            mrawls Meredith Rawls added a comment - Jonathan Sick , would you please review this? Note that the scope changed significantly as I realized how much work it was to manually convert all the docstrings in a python module. This ticket now includes numpydoc conversions for only assembleCoadd.py. 
            mrawls Meredith Rawls made changes -
            Reviewers Jonathan Sick [ jsick ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            swinbank John Swinbank made changes -
            Summary Finish pipe_tasks numpydoc conversion Convert assembleCoadd.py to numpydoc
            Hide
            jsick Jonathan Sick added a comment -

            Thanks for getting it this far! I've got some minor fixes on the PR.

            Show
            jsick Jonathan Sick added a comment - Thanks for getting it this far! I've got some minor fixes on the PR.
            jsick Jonathan Sick made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            mrawls Meredith Rawls added a comment -

            Merged after Jenkins passed on all but centos6, due to a known jointcal issue.

            As per discussion with Jonathan Sick on GitHub, the COPYRIGHT file is a stub and will need to be fully updated later.

            Show
            mrawls Meredith Rawls added a comment - Merged after Jenkins passed on all but centos6, due to a known jointcal issue. As per discussion with Jonathan Sick on GitHub, the COPYRIGHT file is a stub and will need to be fully updated later.
            mrawls Meredith Rawls made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            yusra Yusra AlSayyad made changes -
            Link This issue duplicates DM-13587 [ DM-13587 ]

              People

              Assignee:
              mrawls Meredith Rawls
              Reporter:
              mrawls Meredith Rawls
              Reviewers:
              Jonathan Sick
              Watchers:
              Jonathan Sick, Meredith Rawls
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.