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

Speed up doc builds #1470

Closed
1 of 2 tasks
huard opened this issue Sep 8, 2023 · 3 comments · Fixed by #1476
Closed
1 of 2 tasks

Speed up doc builds #1470

huard opened this issue Sep 8, 2023 · 3 comments · Fixed by #1476
Labels
enhancement New feature or request

Comments

@huard
Copy link
Collaborator

huard commented Sep 8, 2023

Addressing a Problem?

Compiling the documentation takes a really long time. Makes debugging math equations really painful.

Potential Solution

myst-nb has options to cache the notebooks.

Additional context

No response

Contribution

  • I would be willing/able to open a Pull Request to contribute this feature.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@huard huard added the enhancement New feature or request label Sep 8, 2023
@Zeitsperre
Copy link
Collaborator

See also #1439

@huard
Copy link
Collaborator Author

huard commented Sep 8, 2023

Sometime make html goes through all the notebooks, and sometimes it doesn't. Confused as to what triggers this, but has a large impact on build time.

@Zeitsperre
Copy link
Collaborator

This isn't well documented, but it's due to an environment flag-based workaround I added to help with speeding things up:

skip_notebooks = os.getenv("SKIP_NOTEBOOKS")

If your shell has SKIP_NOTEBOOKS defined, the notebooks will not be run. For ReadTheDocs, the notebooks will conditionally be run (auto/always) depending on whether the trigger was coming from a pull request or from main.

This should be made much clearer in the contributing documentation at the very least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

2 participants