Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

DOC: Add all InnerEye/ML docstrings to ReadTheDocs #783

Merged
merged 23 commits into from
Aug 16, 2022

Conversation

peterhessey
Copy link
Contributor

@peterhessey peterhessey commented Aug 8, 2022

Closes #774

Adds all the docstrings in InnerEye/ML to the ReadTheDocs site.

Includes many corrections to docstrings formatting to ensure that they render correctly.

Apologies for the size of PR, but the majority of the line changes are whitespace erro and reStructuredText formatting corrections that I automated to save time. The critical changes for review are all found in the docs/ directory. i.e. the .rst files.

@peterhessey peterhessey changed the title Phessey/update api docs DOC: Add API/docstrings to ReadTheDocs Aug 8, 2022
@peterhessey peterhessey changed the title DOC: Add API/docstrings to ReadTheDocs DOC: Add all InnerEye/ML docstrings to ReadTheDocs Aug 9, 2022
@peterhessey
Copy link
Contributor Author

The sphinx-build process is still dumping a million warnings currently. This will be fixed in a separate PR - I think that a lot of these will disappear when we switch to a new reStructuredText parser.

@peterhessey peterhessey marked this pull request as ready for review August 9, 2022 13:25
Copy link
Contributor

@fepegar fepegar left a comment

Choose a reason for hiding this comment

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

Approved with some comments. Looks great!!

docs/source/conf.py Outdated Show resolved Hide resolved
docs/source/conf.py Outdated Show resolved Hide resolved
docs/source/conf.py Show resolved Hide resolved
docs/source/rst/api/ML/core/lightning.rst Show resolved Hide resolved
@peterhessey peterhessey enabled auto-merge (squash) August 16, 2022 08:35
@peterhessey peterhessey merged commit 59214c2 into main Aug 16, 2022
@peterhessey peterhessey deleted the phessey/update-API-docs branch August 16, 2022 08:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add InnerEye/ML docstrings to ReadTheDocs
3 participants