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

          Parejkoj John Parejko added a comment - Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29185/pipeline

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

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

          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.