FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

CI: Test building docs in parallel on Circle CI by scottshambaugh · Pull Request #28244 · matplotlib/matplotlib · GitHub

Repository navigation

CI: Test building docs in parallel on Circle CI - #28244

Closed
scottshambaugh wants to merge 1 commit into
matplotlib:mainfrom
scottshambaugh:docs_parallel
Closed

scottshambaugh wants to merge 1 commit into
matplotlib:mainfrom
scottshambaugh:docs_parallel

Conversation

scottshambaugh commented May 17, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

PR summary

Just experimenting for now, curious if this speeds things up

For comparison, this docs build took 20m 11s as a baseline on a single core.

PR checklist

scottshambaugh commented May 17, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

With 2 cores the time to build the docs dropped to 18m 9s.

The errors that caused this failure are:

WARNING: the pydata_sphinx_theme extension is not safe for parallel writing
WARNING: doing serial write

This is the relevant issue: pydata/pydata-sphinx-theme#1643. Some users there reported no problems with setting the parallel_write_unsafe flag to False. Will try that next.

Copy link
Copy Markdown
Contributor Author

Ok digging into it more, looks like this was talked about in #27658

The parallel unsafe flag was added in pydata/pydata-sphinx-theme#1642, where this was the behavior:

marks the theme as unsafe for parallel_write. This is based on extensive testing of the SciPy docs build, where the API pages were reliably getting the wrong left sidebar TOC when built in parallel, and getting the correct sidebar TOC when built with a single job. After marking the theme as parallel-write-unsafe, the SciPy docs now end up with the correct sidebar on all pages even when building in parallel (i.e. when pages are read in parallel but written serially)

Worth another look when that all gets fixed but closing for now.

QuLogic mentioned this pull request Jul 26, 2024
1 task done
QuLogic changed the title CI: Test building docs in parallel on Cirrus CI CI: Test building docs in parallel on Circle CI Apr 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL