Repository navigation
[wealth_dynamics] Retarget the JAX cross-reference after the lecture-jax rename - #1082
Conversation
…jax rename QuantEcon/lecture-jax#353 renamed its wealth_dynamics lecture to wealth_dynamics_jax and the republished inventory no longer has the old name, so the {doc} reference here fails and the PDF build, which treats warnings as errors, fails with it on every branch. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
The single-line retarget matches the confirmed lecture-jax rename (wealth_dynamics_jax.md) and the configured jax: intersphinx mapping, with no remaining stale references.
0 open findings
What changed in this PR
This PR fixes a broken intersphinx cross-reference in wealth_dynamics.md. After lecture-jax renamed wealth_dynamics to wealth_dynamics_jax (QuantEcon/lecture-jax#353) and redeployed, the old {doc} target jax:wealth_dynamics no longer resolves, causing PDF builds (where warnings are errors) to fail with "unknown document: 'jax:wealth_dynamics'". The change retargets the link to jax:wealth_dynamics_jax, restoring the "seealso" link to the JAX version of this lecture.
Changes:
- Update the
{doc}intersphinx reference fromjax:wealth_dynamicstojax:wealth_dynamics_jaxto match the renamed lecture-jax document.
| File | Description |
|---|---|
lectures/wealth_dynamics.md |
Retargets the JAX "seealso" cross-reference to the renamed wealth_dynamics_jax document so builds resolve the link. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
✅ Translation sync completed (zh-cn)Target repo: QuantEcon/lecture-python.zh-cn
|
📖 Netlify Preview Ready!Preview URL: https://pr-1082--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
QuantEcon/lecture-jax#353 renamed twelve lectures this morning, including
wealth_dynamicstowealth_dynamics_jax, and lecture-jax's publish-2026oct09 release put the new names into the intersphinx inventory at 01:02 UTC. The{doc}cross-reference inwealth_dynamics.mdstill used the old name, so every build on this repo since then fails at the PDF step with "unknown document: 'jax:wealth_dynamics'" (warnings are errors there). #1081's preview failed this way. The publish-2026oct09 run here succeeded only because its PDF step ran before the lecture-jax site had actually redeployed, so the next publish would fail without this change.This retargets the reference to
jax:wealth_dynamics_jax. It is the only link from this repo into lecture-jax; the rename PR's search for cross-series links did not catch it because it is an intersphinx reference rather than a plain lecture name.🤖 Generated with Claude Code