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

Convert afw.cameraGeom to numpydoc

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • afw
    • 1
    • AP S19-1, AP S19-2
    • Alert Production

    Description

      Convert all Doxygen-formatted docstrings in afw.cameraGeom to Numpydoc style, and move corresponding topic documentation to doc/.

      Attachments

        Activity

          No builds found.
          krzys Krzysztof Findeisen created issue -
          krzys Krzysztof Findeisen made changes -
          Field Original Value New Value
          Epic Link DM-16722 [ 235355 ]
          Parejkoj John Parejko made changes -
          Assignee John Parejko [ parejkoj ]
          Parejkoj John Parejko made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          Parejkoj John Parejko added a comment - Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29185/pipeline
          Parejkoj John Parejko made changes -
          Story Points 1

          rowen: Thanks for volunteering.

          The commits are atomic; you probably want to look at them separately.

          Parejkoj John Parejko added a comment - rowen : Thanks for volunteering. The commits are atomic; you probably want to look at them separately.
          Parejkoj John Parejko made changes -
          Reviewers Russell Owen [ rowen ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          rowen Russell Owen added a comment - - edited

          Pull request: https://github.com/lsst/afw/pull/426

          I took a first pass and requested some changes.

          rowen Russell Owen added a comment - - edited Pull request: https://github.com/lsst/afw/pull/426 I took a first pass and requested some changes.
          swinbank John Swinbank made changes -
          Sprint AP S19-1 [ 825 ] AP S19-1, AP S19-2 [ 825, 830 ]

          rowen: I've cleaned up per your comments and converted the `cameraGeom.dox` file to .rst and fixed its links. Anything else?

          Parejkoj John Parejko added a comment - rowen : I've cleaned up per your comments and converted the `cameraGeom.dox` file to .rst and fixed its links. Anything else?
          rowen Russell Owen added a comment -

          Looks great! Just a few small requested changes on github (most notably, please make the class names links in the new .rst file)

          rowen Russell Owen added a comment - Looks great! Just a few small requested changes on github (most notably, please make the class names links in the new .rst file)
          rowen Russell Owen made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Parejkoj John Parejko added a comment - Thanks for the review! New Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29251/pipeline
          Parejkoj John Parejko added a comment -

          Merged and done.

          Parejkoj John Parejko added a comment - Merged and done.
          Parejkoj John Parejko made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

          People

            Parejkoj John Parejko
            krzys Krzysztof Findeisen
            Russell Owen
            John Parejko, Krzysztof Findeisen, Russell Owen
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.