Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: meas_extensions_photometryKron
-
Labels:None
-
Story Points:0.5
-
Epic Link:
-
Sprint:DRP X16-3
-
Team: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
- blocks
-
- Done
Activity
Sure, I'll take a look at it.
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?
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.
Merged.
Hey Pim, could you take a look at this please? Should be pretty straightforward. PR here.