Skip to content

Add @param ppower Doxygen documentation to Cylinder.H#206

Merged
The9Cat merged 2 commits intoSphericalExactfrom
copilot/sub-pr-199
Feb 25, 2026
Merged

Add @param ppower Doxygen documentation to Cylinder.H#206
The9Cat merged 2 commits intoSphericalExactfrom
copilot/sub-pr-199

Conversation

Copy link

Copilot AI commented Feb 25, 2026

ppower was parsed from YAML config and registered in valid_keys but had no corresponding Doxygen @param entry in the Cylinder class header, unlike other parameters such as mtype and bias.

Changes

  • src/Cylinder.H: Added @param ppower documentation entry after mtype in the class Doxygen block, describing it as the power-law index for the Power spherical deprojection model used in basis conditioning, with its default value (2.0) and the condition under which it applies (mtype set to Power).

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: The9Cat <25960766+The9Cat@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on spherical basis for density deprojection Add @param ppower Doxygen documentation to Cylinder.H Feb 25, 2026
@The9Cat The9Cat marked this pull request as ready for review February 25, 2026 19:15
@The9Cat The9Cat merged commit 97bdeb7 into SphericalExact Feb 25, 2026
@The9Cat The9Cat deleted the copilot/sub-pr-199 branch February 25, 2026 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants