🌐 [translation-sync] [FIX] Add pip install cells for quantecon and jax in lectures that were missing them#208
Merged
Conversation
mmcky
added
action-translation
✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Translation Quality ReviewVerdict: PASS | Model: claude-sonnet-5 | Date: 2026-07-26 📝 Translation Quality
Summary: 本次翻译整体质量很高，专业术语准确且与术语表高度一致，数学公式与代码结构完整保留，未发现语法错误。少数句子的流畅度可以进一步打磨，但均为轻微问题，不影响整体理解和准确性。 术语翻译高度符合参考术语表，如“最大似然估计”“贝尔曼算子”“平稳分布”“牛顿法”等专业词汇准确一致 数学公式、代码块及MyST指令格式完整保留，未出现语法错误 长难句的拆分与重组处理得当，读起来符合学术中文的表达习惯 标题元数据（translation部分）按规范正确添加，章节标题对应准确 Suggestions:
🔍 Diff Quality
Summary: The translation sync correctly mirrors positional changes (pip install additions) across all shared files, but the full addition of olg_adaptive_money.md as a brand-new file raises scope concerns since it wasn't a small targeted change like the other files. Issues:
This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
Pull request overview
This translation-sync PR updates the zh-cn lecture notebooks to ensure required runtime dependencies are installed via hidden pip install cells (primarily jax, plus quantecon where relevant), aligning the translated lectures with upstream execution requirements.
Changes:
- Added missing hidden
pip installcells forjax(andquanteconin relevant lectures) so notebooks have explicit dependency setup. - Added a new translated lecture file
lectures/olg_adaptive_money.md(includespip install quantecon). - Updated translation state metadata (
.translate/state/*.yml) to reflect the new upstream source SHA, sync date, and tool version.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lectures/olg_adaptive_money.md | New translated lecture; includes hidden pip install quantecon for required imports. |
| lectures/newton_method.md | Adds hidden pip install jax cell; minor notebook metadata/formatting normalization. |
| lectures/mle.md | Adds hidden pip install jax cell; normalizes notebook metadata header. |
| lectures/kesten_processes.md | Extends install cell to include jax alongside existing dependencies; minor formatting cleanup. |
| lectures/jv.md | Adds hidden pip install jax cell for JAX-based implementation sections. |
| lectures/inventory_dynamics.md | Adds hidden pip install jax cell for JAX-based simulation code. |
| lectures/ifp_advanced.md | Updates install cell to include jax alongside quantecon. |
| lectures/career.md | Updates install cell to include jax alongside quantecon. |
| .translate/state/olg_adaptive_money.md.yml | New translation state entry for the added lecture. |
| .translate/state/newton_method.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
| .translate/state/mle.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
| .translate/state/kesten_processes.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
| .translate/state/jv.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
| .translate/state/inventory_dynamics.md.yml | Updates translation state (source SHA / sync timestamp / tool version). |
| .translate/state/ifp_advanced.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
| .translate/state/career.md.yml | Updates translation state (source SHA / sync timestamp / mode / tool version). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This was referenced Jul 27, 2026
mmcky
added a commit
that referenced
this pull request
Jul 27, 2026
…he undelivered lecture, backfill the bib Three fixes, all consequences of the run that produced this PR being partial (see QuantEcon/action-translation#222). 1. Merge conflict on olg_adaptive_money. Both this PR and #208 created lectures/olg_adaptive_money.md and its state file from scratch. Keep main's version, which #208 landed: it is built from the newer source SHA (a4fbd86) and so carries the pip-install cell, and it matches the English source structurally at 62 fenced directives to 62. This branch's copy predates #1006 and is one directive short. 2. learning_approximation was never delivered. The run committed a _toc.yml entry for it but failed structural parity on the file, so the entry pointed at nothing and the build failed with "toctree contains reference to nonexisting document", plus four unknown-document cross-references. Drop the _toc.yml entry and remove the four {doc} references from bounded_rationality.md and genetic_classifier.md. The lecture stays untranslated and is tracked separately; the references return when it is seeded. 3. quant-econ.bib was missing 21 keys cited by the new lectures, fatal under -W. Replace the file wholesale from the English source rather than backfilling the cited subset — the target bib is a strict subset of source (0 target-only keys), this is the third such backfill, and the wholesale copy also closes the three latent keys that are not yet cited. Mechanism tracked in QuantEcon/action-translation#117. Verified: all 355 citations resolve, every _toc.yml entry has a file, every lecture is reachable from _toc.yml, no dangling {doc} references. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated Translation Sync
This PR contains automated translations from QuantEcon/lecture-python.myst.
Source PR
#1006 - [FIX] Add pip install cells for quantecon and jax in lectures that were missing them
Files Added
lectures/olg_adaptive_money.md.translate/state/olg_adaptive_money.md.ymlFiles Updated
lectures/career.md.translate/state/career.md.ymllectures/ifp_advanced.md.translate/state/ifp_advanced.md.ymllectures/inventory_dynamics.md.translate/state/inventory_dynamics.md.ymllectures/jv.md.translate/state/jv.md.ymllectures/kesten_processes.md.translate/state/kesten_processes.md.ymllectures/mle.md.translate/state/mle.md.ymllectures/newton_method.md.translate/state/newton_method.md.ymlDetails
This PR was created automatically by the translation action.