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

Restructure Documentation #1679

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

MarcelKoch
Copy link
Member

This PR splits the documentation into a part that is generated by sphinx and one that is generated by doxygen.
The doxygen part will cover only the reference API documentation, and anything else will be left to sphinx. The full separation is not done in this PR.

Changes:

  • adds sphinx configuration
  • make doxygen available for sphinx
  • simplify doxygen setup
    • only the usr doc is available, the dev doc is removed. IMO there is no need for a dev doc, since we can expect developers to look into the Ginkgo source code and discover the documentation there.
  • fix some existing documentation (grouping, markdown files)
  • use read-the-docs instead of GH pages to deploy documentation

TODO:

  • finalize the read-the-docs namespace

@MarcelKoch MarcelKoch self-assigned this Sep 12, 2024
@ginkgo-bot ginkgo-bot added reg:build This is related to the build system. reg:documentation This is related to documentation. mod:core This is related to the core module. type:solver This is related to the solvers type:preconditioner This is related to the preconditioners type:matrix-format This is related to the Matrix formats type:factorization This is related to the Factorizations labels Sep 12, 2024
@MarcelKoch
Copy link
Member Author

Link to the documentation at RTD: https://ginkgo-test.readthedocs.io/en/read-the-docs/

@MarcelKoch MarcelKoch force-pushed the read-the-docs branch 5 times, most recently from 251d914 to 02a4a81 Compare September 13, 2024 09:02
This does the following things:
- add sphinx to create non-reference/api documentation
- move doxygen stuff into a subdirectory of the sphinx configuration
- enable read-the-docs to host documentation
The linop group is often added with other groups. These other groups are mostly subgroups of linop, so there is no need to specify it twice.
@MarcelKoch MarcelKoch added the 1:ST:ready-for-review This PR is ready for review label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1:ST:ready-for-review This PR is ready for review mod:core This is related to the core module. reg:build This is related to the build system. reg:documentation This is related to documentation. type:factorization This is related to the Factorizations type:matrix-format This is related to the Matrix formats type:preconditioner This is related to the preconditioners type:solver This is related to the solvers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants