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

UNIQUE Documentation Deployment to GitHub Pages #3

Merged
merged 7 commits into from
Sep 20, 2024

Conversation

mtdhuynh
Copy link
Collaborator

@mtdhuynh mtdhuynh commented Sep 20, 2024

UNIQUE Documentation Deployment to GitHub Pages

This PR adds a GitHub Actions workflow to deploy the (HTML) documentation to GitHub Pages. The target domain will be the default provided by GitHub - username.github.io/repository (i.e., novartis.github.io/unique). At a later point in time, it will be possible to move the documentation to a custom domain (see the docs).

Major Changes

  • Added docs.yml with GitHub Actions workflow to build and deploy documentation to GitHub Pages.

Minor Changes

  • Updated advanced installation instructions to only use conda instead of mamba in both documentation and in the Makefile. As of conda > 23.10, libmamba is the default solver, removing the need to specifically install mamba to use it. Reverting to conda to make it more user-friendly/accessible.

To Do

  • Troubleshoot the default actions/cache action for Jupyter notebook execution caching.

Currently, documentation build takes ~25 minutes due to having to execute the example notebooks. However, the jupyter-cache add-in in myst-nb allows to cache and reuse the execution outputs if the notebooks have not been modified since the last build.

Caching and restoring the execution outputs using the actions/cache action creates a situation whereby the make docs command does not run and says make: 'docs' is up to date.

Being able to cache and reuse the execution outputs can speed up documentation build if notebooks are untouched by current changes. Not critical, but nice to have.

@mtdhuynh
Copy link
Collaborator Author

@gscebba have a look and let me know!

@gscebba gscebba merged commit 8b1999e into Novartis:main Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants