-
Notifications
You must be signed in to change notification settings - Fork 52
Initial Documentation #51
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
Conversation
This negates the need to manually configure advanced readthedocs options through the web interface. It also negates the need for docs/requirements.txt, which can instead use extras_require options from setup.cfg
Built version can be seen here: https://philipstarkey-labscript-utils.readthedocs.io/en/latest/ |
Proposals:
I get a different result when running |
I don't have any intermediate build results in the api_reference folder...unless you are referring to the .rst files? I made these by hand. Didn't realise that sphinx-apidoc could auto generate them....I'll need to look into how flexible that is. I was expecting that we might write some introductory text and or need to explicitly state the signatures of some methods (e.g. |
Oh, I see.
Yep, more experimentation is required before using
Relevant to this:
|
It might be prudent to merge/add changes to conf.py from labscript-suite/labscript-suite#43. |
I have incorporated changes from labscript-suite/labscript-suite#43. Happy to merge now? |
I think there's one more: # Customize the html_theme
html_theme_options = {'navigation_depth': 3} |
Fine by me if you don't mind bd0d527 changes (feel free to force push if you don't). |
This PR includes initial documentation for labscript-utils
The Sphinx conf.py file is a superset of the rpanderson/lyse and labscript-suite/labscript-suite conf.py files, with a bit extra that should allow it to be used across multiple projects with minimal adjustments.
This PR also includes an updated .gitignore (also a superset of labscript-suite/labscript-suite and labscript-suite/labscript)
I have also created a readthedocs.yaml file which provides configuration information to reathedocs that should override the advanced settings in the web interface (meaning these settings are now version controlled). The advantage of this format is that you can specify requirements as extras_requires within setup.cfg and tell readthedocs to install the package using that options (e.g. it runs
pip install .[docs]
).Some very basic initial documentation is included (a couple of autodocs). It needs fleshing out more, but the purpose of this PR is to get the initial config into the repository so that documentation can become a collaborative effort.
If this looks good to people, I aim to rapidly duplicate it across other repositories that don't yet have docs or people working on docs (which I think is runmanager, runviewer, blacs, and labscript-devices).