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

Documentation: Replace Sphinx Build with Jupyter Book #400

Merged
merged 23 commits into from
Nov 13, 2024

Conversation

RHammond2
Copy link
Collaborator

@RHammond2 RHammond2 commented Nov 5, 2024

Documentation: Update documentation site to Jupyter Book

This PR addresses the previously discussed process to convert the HOPP documentation to a Jupyter Book-powered site with the Jupyter Book theme. In addition to converting the build steps, much of the infrastructure has been simplified to reduce the highly nested API layout (mimicked hopp/) to one that is easier to maintain over a longer period of time.

Related issue

Resolves #399

Impacted areas of the software

  • docs/: New top-level files for the jupyter book specification and
  • docs/api/: The API folder replaces the older structure that mimics the hopp/ layout to simplify the documentation layout. Essentially there were many files dedicated to using autodoc to build a page for single (or two) classes that did not make much sense from an end product standpoint.
  • readthedocs.yaml: Converts to the jupyter book build steps
  • .github/workflows/: Two new workflows have been added: one for testing the documentation build on PRs, and another for publishing the documentation site to GitHub Pages

Additional supporting information

I have pinned additionally pinned Shapely at >= 2 since our use of their API changed in a previous release without pinning HOPP's minimum allowed version causing some autodoc build errors for me.

Test results, if applicable

N/A.

@RHammond2 RHammond2 added the documentation Improvements or additions to documentation label Nov 5, 2024
@RHammond2 RHammond2 marked this pull request as ready for review November 8, 2024 00:05
@johnjasa johnjasa merged commit 6d094d2 into NREL:develop Nov 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants