Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pace documentation (https://ai2cm.github.io/pace/) #67

Open
xyuan opened this issue Feb 13, 2024 · 6 comments
Open

pace documentation (https://ai2cm.github.io/pace/) #67

xyuan opened this issue Feb 13, 2024 · 6 comments

Comments

@xyuan
Copy link

xyuan commented Feb 13, 2024

there is markdown pace documentation https://ai2cm.github.io/pace/, but we are unable to edit, could this page be part of our github repo, so we can add more documents to it?

@FlorianDeconinck
Copy link
Collaborator

This is generated code from the docs folder. Command line is still in the makefile but might need adjustements.
Docs content is pre-refactor and will also require adjustments.

Deploying to Github Pages can be automatized on PR merge and/or Release: https://github.com/actions/deploy-pages

@FlorianDeconinck
Copy link
Collaborator

The main thing is that right now the docs folder in Pace has documentation for Pace but also pyFV3, pySHiELD and NDSL

@fmalatino
Copy link
Contributor

I think we could also add more documentation to this as well to help new users/developers get up to speed. This could include testing methods, descriptions of the functionality, etc. I think this issue can be taken on as a collaborative effort, as new features are added this will grow with them.

@xyuan
Copy link
Author

xyuan commented Feb 14, 2024

we could think about the doxygen, but it requires some additional work on the source code. I am also thinking about adding some theoretical, and algorithm background on the document page too.

@FlorianDeconinck
Copy link
Collaborator

We have sphinx setup, no need for doxygen

@xyuan
Copy link
Author

xyuan commented Feb 14, 2024

that is great, though.
would this document be separated for pace, fv3, shield, or just pace is enough

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

No branches or pull requests

3 participants