-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update Sphinx config #415
Update Sphinx config #415
Conversation
Update the requirements files, especially for the documentation. Fix up the conf.py file for configuring the Sphinx build. Add explicitly the AutoAPI templates. Update the "homepage" index file to include a nicer ToC.
These tests are absolutely HORRIBLE!!!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR addresses several (unrelated?) issues which could have been resolved in separate PRs. My suggestion is to approve this PR now, and create follow-up tasks.
Yeah. Bad practice, sorry. Some are a bit unrelated, most are indirectly related though. |
@daniel-sintef I have created #416 and #417 as follow-up tasks/issues. This should cover it for now? |
…e-clean-sphinx-config
…e-clean-sphinx-config
Closes #400
This is a rather lengthy PR and may require several rounds of review.
The goal is to update the Sphinx configuration (
conf.py
) file and remove (or reduce as much as possible) the build warnings/errors. As the main part of these warnings are currently emitted from the auto-generated API documentation, this means updating the doc-strings where necessary.The
triplestore
module has been stripped of everything except theunits
files.Currently, the warnings/errors present pertain to these parts:
share/dlite/mapping-plugins/
.dlite
anddlite.dlite
.Concerning point 1, this is due to duplications in some methods or similar that are defined both in the C code and included Python files. Or it may be due to overloaded functions, something that Sphinx and AutoAPI cannot handle.
For point 2, this will be solved by #412.
The multiple cross-references in point 3 is something that may be fixable in this PR. I need to investigate this further.