ginkgo icon indicating copy to clipboard operation
ginkgo copied to clipboard

Restructure Documentation

Open MarcelKoch opened this issue 8 months ago • 5 comments

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

This is a continuation from #1679. Compared to the previous PR, this has a cleaner integration of doxygen into the documentation build by sphinx.

Link to generated docs: https://ginkgo-test.readthedocs.io/doxysphinx/index.html

Todo:

  • move to proper RTD space

MarcelKoch avatar Mar 20 '25 16:03 MarcelKoch

Plain program under the example shows empty block. https://ginkgo-test.readthedocs.io/doxysphinx/doxygen/html/simple_solver.html

yhmtsai avatar Mar 24 '25 09:03 yhmtsai

Plain program under the example shows empty block. https://ginkgo-test.readthedocs.io/doxysphinx/doxygen/html/simple_solver.html

Should be fixed now.

MarcelKoch avatar Mar 24 '25 15:03 MarcelKoch

for code ```c++ will give a ++ in the beginning on the section https://ginkgo-test.readthedocs.io/doxysphinx/doxygen/html/group__Executor.html image

yhmtsai avatar Mar 25 '25 12:03 yhmtsai

for code ```c++ will give a ++ in the beginning on the section

It's the same for our current documentation: https://ginkgo-project.github.io/ginkgo-generated-documentation/doc/develop/group__Executor.html#gae20be793db14dfd6249a315281201550. I want to keep changes to the actual text of the documentation out of this PR, so I would not fix this right now.

MarcelKoch avatar Mar 25 '25 15:03 MarcelKoch

Error: The following files need to be formatted:

core/solver/ir.cpp
include/ginkgo/core/multigrid/pgm.hpp

You can find a formatting patch under Artifacts here or run format! if you have write access to Ginkgo

ginkgo-bot avatar Mar 25 '25 16:03 ginkgo-bot