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

Provide minimal documentation for meas_extensions_photometryKron

    XMLWordPrintable

Details

    • 0.5
    • DRP X16-3
    • Data Release Production

    Description

      Please provide a minimal level of documentation for meas_extensions_photometryKron, to include:

      • A doc directory with the usual content so that docstrings get generated by Doxygen;
      • A package overview;
      • All docstrings should be appropriate for parsing by Doxygen (ie, should start with """! where necessary).

      Attachments

        Issue Links

          Activity

            Hey Pim, could you take a look at this please? Should be pretty straightforward. PR here.

            swinbank John Swinbank added a comment - Hey Pim, could you take a look at this please? Should be pretty straightforward. PR here .

            Sure, I'll take a look at it.

            pschella Pim Schellart [X] (Inactive) added a comment - Sure, I'll take a look at it.

            Looks fine, if indeed quite minimal. Two minor remarks / questions in PR.

            Also I get a Doxygen warning warning: unable to resolve reference to `measBaseIntroduction' for \ref command when building.
            But I assume this disappears when building the full stack?

            pschella Pim Schellart [X] (Inactive) added a comment - Looks fine, if indeed quite minimal. Two minor remarks / questions in PR. Also I get a Doxygen warning warning: unable to resolve reference to `measBaseIntroduction' for \ref command when building. But I assume this disappears when building the full stack?

            Thanks Pim.

            Your tweak to the English was (of course) correct, and I think your other comments were addressed on the PR.

            swinbank John Swinbank added a comment - Thanks Pim. Your tweak to the English was (of course) correct, and I think your other comments were addressed on the PR.

            Merged.

            swinbank John Swinbank added a comment - Merged.

            People

              swinbank John Swinbank
              swinbank John Swinbank
              Pim Schellart [X] (Inactive)
              John Swinbank, Pim Schellart [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.