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

Overhaul documentation and use Book theme #600

Merged
merged 19 commits into from
Dec 12, 2022

Conversation

peanutfun
Copy link
Member

@peanutfun peanutfun commented Dec 8, 2022

Changes proposed in this PR:

  • Switch to Book theme
  • Add requirement specifications for building the documentation locally
  • Add doc/README.md with instructions on how to build the documentation
  • Add doc/create_env_doc.dockerfile for building the documentation environment from scratch
  • Overhaul documentation with
    • New sub-structure for tutorials
    • Logos and favicon
    • Addition of missing tutorials and guides
    • Inclusion of Markdown documents from repository
    • Explanation on how to add requirements/env_developer.yml to the climada_env

This PR resolves #592 #599 #603

PR Author Checklist

PR Reviewer Checklist

This enables building the docs locally using the climada_env.
* Load Sphinx extensions for parsing Markdown.
* Fix exclude patterns for build directory.
* Update project descriptors.
* Add logo and favicon to docs.
* Switch to Book HTML theme.
* Add subgroups for tutorials.
* Add missing tutorials.
* Add symlinks to top-level markdown files for including them in the
  docs.
@peanutfun peanutfun changed the base branch from main to develop December 8, 2022 11:21
@peanutfun
Copy link
Member Author

This is how it looks like.

Screenshot 2022-12-08 at 12 18 30

* Move symlinks from /doc to /doc/misc.
* Add doc/README.md with instructions for building the documentation.
* Update installation instructions to include the developer
  requirement specifications.
@peanutfun peanutfun marked this pull request as ready for review December 8, 2022 15:11
@chahank
Copy link
Member

chahank commented Dec 8, 2022

Great! I would suggest to:

  • Re-dd the step by step installation guide
  • Add a reference to CLIMADA petals at the top

* Add step-by-step installation tutorial again.
* Add hint for switching versions on Readthedocs.
* Fix links on index page.
@peanutfun
Copy link
Member Author

Latest version. Note that the "hint" will only appear on ReadTheDocs, not when building the docs on your local machine.

Screenshot 2022-12-09 at 11 11 20

@peanutfun peanutfun linked an issue Dec 9, 2022 that may be closed by this pull request
* Create a Dockerfile for building a pristine conda environment for
  extracting the doc requirements.
* Add instructions on how to use it to doc/README.md.
* Add doc/README.md to the Sphinx documentation.
@peanutfun peanutfun linked an issue Dec 12, 2022 that may be closed by this pull request
@peanutfun
Copy link
Member Author

@emanuel-schmid Could you give this a look? It's now quite a bit but it resolves three issues at once 💪

You will find the preview here: https://climada-python--600.org.readthedocs.build/en/600/

I already incorporated the suggestions by @chahank. This is ready to go!

@emanuel-schmid
Copy link
Collaborator

🤩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants