Skip to content

Fix failing Read the Docs builds: raise myst-nb execution timeout#495

Merged
basnijholt merged 1 commit into
mainfrom
fix-docs-build-timeout
Jun 10, 2026
Merged

Fix failing Read the Docs builds: raise myst-nb execution timeout#495
basnijholt merged 1 commit into
mainfrom
fix-docs-build-timeout

Conversation

@basnijholt

Copy link
Copy Markdown
Member

Problem

Read the Docs builds have been failing intermittently (e.g. build 33082181 for latest, plus builds for #493 and several PR previews) with:

nbclient.exceptions.CellTimeoutError: A cell timed out while it was being executed, after 180 seconds.
myst_nb.core.execute.base.ExecutionError: .../docs/source/logo.md

The logo.md notebook regenerates the animated docs logo on every build (the .webm is not committed): it runs a Learner2D, renders 360 PNG frames, and encodes them with libvpx-vp9. On Read the Docs builders this takes close to 3 minutes, so it lands right around the 180 s nb_execution_timeout — sometimes passing, sometimes not. The build history shows the same flakiness: identical doc content alternates between success and failure depending on builder speed.

Fix

Raise nb_execution_timeout from 180 to 600 seconds in docs/source/conf.py, giving the logo cell (and any other slow notebook) comfortable headroom without affecting builds that finish quickly.

Verification

The RTD build for this PR's preview version should be the real test — all recent failures reproduce only on RTD builders.

The logo animation notebook (docs/source/logo.md) renders 360 frames and
encodes them with libvpx-vp9, which takes close to 3 minutes on Read the
Docs builders. The 180 s nb_execution_timeout was intermittently exceeded,
causing docs builds to fail with CellTimeoutError.
@basnijholt basnijholt merged commit 1bf65d9 into main Jun 10, 2026
18 checks passed
@basnijholt basnijholt deleted the fix-docs-build-timeout branch June 10, 2026 20:54
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.

1 participant