Skip to content

[main] ccpp-physics SciDocs to github pages#1207

Merged
grantfirl merged 155 commits intoNCAR:mainfrom
hertneky:scidocs_hsd_aqua
Apr 16, 2026
Merged

[main] ccpp-physics SciDocs to github pages#1207
grantfirl merged 155 commits intoNCAR:mainfrom
hertneky:scidocs_hsd_aqua

Conversation

@hertneky
Copy link
Copy Markdown
Collaborator

@hertneky hertneky commented Mar 27, 2026

Description of Changes:

Move SciDocs to github pages. The landing page can be viewed here, with links to the various document versions.

Will need to update branch to main in doxygen workflow before merging.

Tests Conducted:

github actions

Dependencies:

no

Documentation:

Issue (optional):

Contributors (optional):

Built off @mzhangw original PR - thanks!

Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I probably need another pass through the documentation, but this looks really good!

@hertneky
Copy link
Copy Markdown
Collaborator Author

hertneky commented Apr 9, 2026

Looks good with minor suggestions on the mainpage and directory structure. And it is also noticed that argument table is not shown correctly (see here) in the current doxygen output.

@mzhangw The argument tables should now all be working. I've also made the other updates you suggested. This should be good to go now.
For now, to get the argument tables to work, I use the scm host model, but in the future, we may want to eliminate host model/framework dependencies to build the physics docs. Unfortunately, I don't have the resources to do that atm.

@hertneky hertneky requested a review from mzhangw April 9, 2026 16:21
@hertneky
Copy link
Copy Markdown
Collaborator Author

hertneky commented Apr 9, 2026

I probably need another pass through the documentation, but this looks really good!

@climbfuji And now the argument tables should be populating, so it's ready for a more in-depth review whenever!

@climbfuji
Copy link
Copy Markdown
Collaborator

I probably need another pass through the documentation, but this looks really good!

@climbfuji And now the argument tables should be populating, so it's ready for a more in-depth review whenever!

Indeed, they do! Thanks for that. I'll go ahead and approve. I didn't review every single page of it, but poked around and it all looked great.

@grantfirl grantfirl changed the title ccpp-physics SciDocs to github pages [main] ccpp-physics SciDocs to github pages Apr 10, 2026
@grantfirl
Copy link
Copy Markdown
Collaborator

@hertneky Please go ahead and update the github doxygen workflow to use the main branch so that we can merge this.

@hertneky
Copy link
Copy Markdown
Collaborator Author

@hertneky Please go ahead and update the github doxygen workflow to use the main branch so that we can merge this.

done

@grantfirl grantfirl merged commit 6203bc2 into NCAR:main Apr 16, 2026
3 checks passed
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.

4 participants