chore: bump the pinned theme to v2.5.0 and restore the short_path graph cell - #72
Conversation
…ph cell The site pinned quantecon-theme.mystmd v2.3.0 and served 60 KaTeX font 404s (theme#138, fixed in v2.3.1 and completed in v2.4.0). v2.5.0 also brings the Sphinx-matching typography, the Colab-only launcher and the toolbar accessibility fixes. See #71. The %%file graph.txt cell moves back inside the exercise: the nested code-cell kernel defect it was working around (theme#117) shipped in v2.3.1, and the workaround's own comment said to revert it then. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
❌ Deploy Preview for beautiful-dodol-cb9543 failed.
|
There was a problem hiding this comment.
🟢 Approval recommended
The changes are small, consistent, and keep directive structure/config valid while aligning with the stated upstream theme fix.
Pull request overview
This PR updates the MyST site’s pinned QuantEcon theme to v2.5.0 (to pick up upstream KaTeX asset URL fixes and UI/accessibility improvements) and restores the short_path exercise structure by moving the graph-data %%file graph.txt cell back inside the exercise now that nested code cells are expected to register with the kernel.
Changes:
- Bump the pinned
site.templateURL from QuantEcon theme v2.3.0 to v2.5.0. - Remove the prior workaround comment and relocate the
graph.txtwriting code cell back within theshort_pathexercise block.
File summaries
| File | Description |
|---|---|
| lectures/short_path.md | Moves the graph-data code-cell back inside the exercise and removes the obsolete workaround note. |
| lectures/myst.yml | Updates the pinned theme template URL to QuantEcon theme v2.5.0. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
📖 Netlify Preview Ready! Preview URL: https://pr-72--beautiful-dodol-cb9543.netlify.app (cf4c2c1) ✨ Browse the preview at the URL above. |
Bumps the pinned lecture theme from v2.3.0 to v2.5.0 and moves the
short_pathgraph-data cell back inside its exercise, now that the theme carries the fix the workaround was for. See #71.What is broken on the live site
The deployed KaTeX stylesheet references its 60 font files under
/myst_assets_folder/_assets/, a path a static build never creates, so every one of them 404s and every maths glyph falls back to a system font. Re-verified against the live site on 2026-09-07. The cause is QuantEcon/quantecon-theme.mystmd#138, fixed in v2.3.1 and completed in v2.4.0 (four route and shared-chunk stylesheets the first fix missed). This site is the only mystmd consumer, and no bot bumps asite.templateURL, which is why the pin fell two releases behind.What the bump brings
The launcher change is safe here: this site enables JupyterLite and never configured the Private JupyterHub option that v2.5.0 removes. The typography changes are visible, and are the direction the theme's parity work is taking every lecture site.
The
short_pathrevert#63 moved the
%%file graph.txtcell out of the exercise because, on v2.3.0, a code cell nested in any directive rendered a run button that did nothing and the solution then failed withFileNotFoundError. Its comment said to move the cell back once QuantEcon/quantecon-theme.mystmd#117 shipped. It shipped in v2.3.1. The cell returns to where it sits inlecture-python-introand the other five repos that carry this exercise, and the comment goes.Verification
myst build --htmlagainst the v2.5.0 template completes./myst_assets_folderURLs.Not touched: #56 (root-relative
myst-theme.css, favicon and service-worker URLs ignoringBASE_URL), which is a defect in this repository's own configuration.🤖 Generated with Claude Code