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

Convert afw.cameraGeom to numpydoc

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
    • Story Points:
      1
    • Sprint:
      AP S19-1, AP S19-2
    • Team:
      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 ]
          Show
          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
          Hide
          Parejkoj John Parejko added a comment -

          Russell Owen: Thanks for volunteering.

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

          Show
          Parejkoj John Parejko added a comment - Russell Owen : 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 ]
          Hide
          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.

          Show
          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 ]
          Hide
          Parejkoj John Parejko added a comment -

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

          Show
          Parejkoj John Parejko added a comment - Russell Owen : I've cleaned up per your comments and converted the `cameraGeom.dox` file to .rst and fixed its links. Anything else?
          Hide
          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)

          Show
          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 ]
          Show
          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
          Hide
          Parejkoj John Parejko added a comment -

          Merged and done.

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

            People

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

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins Builds

                No builds found.