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

docs(graphs): Refactor anemoi-graphs documentation #49

Merged
merged 65 commits into from
Jan 24, 2025
Merged

Conversation

JPXKQX
Copy link
Member

@JPXKQX JPXKQX commented Dec 20, 2024

This pull request will address a refactor of the documentation structure to improve user experience.


📚 Documentation preview 📚: https://anemoi-training--49.org.readthedocs.build/en/49/


📚 Documentation preview 📚: https://anemoi-graphs--49.org.readthedocs.build/en/49/


📚 Documentation preview 📚: https://anemoi-models--49.org.readthedocs.build/en/49/

@FussyDuck
Copy link

FussyDuck commented Jan 7, 2025

CLA assistant check
All committers have signed the CLA.

@HCookie
Copy link
Member

HCookie commented Jan 21, 2025

I'm keen to see this PR once the ReadTheDocs workflow is back, I'll review it then.

graphs/CHANGELOG.md Show resolved Hide resolved
graphs/docs/cli/introduction.rst Show resolved Hide resolved
graphs/docs/usage/getting_started.rst Show resolved Hide resolved
graphs/docs/overview.rst Show resolved Hide resolved
@anaprietonem anaprietonem self-requested a review January 24, 2025 12:50
Copy link
Contributor

@anaprietonem anaprietonem left a comment

Choose a reason for hiding this comment

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

Mario addressed all my previous comments. The preview of the docs is now correctly rendered and looks good so I am approving this PR

@JPXKQX JPXKQX merged commit 29942b6 into main Jan 24, 2025
15 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation graphs training
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants