🌐 [translation-sync] [likelihood_bayes] Update lecture according to the style guide - #240
🌐 [translation-sync] [likelihood_bayes] Update lecture according to the style guide#240mmcky wants to merge 6 commits into
Conversation
✅ 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-08-05 📝 Translation Quality
Summary: translation整体准确、术语规范,格式基本保持完整,但在代码示例中存在与英文原文不一致的randomness API使用问题(缺少rng对象定义),这可能导致读者运行代码时产生错误,属于格式/技术准确性问题,建议修正。语言流畅度总体良好,仅有少量表达可进一步优化。 数学公式、LaTeX表达式与代码块基本完整保留,未破坏原始格式 专业术语(如“似然比过程”“贝叶斯定律”“KL散度”等)翻译准确且与术语表高度一致 长难句的拆分和结构调整基本符合中文学术写作习惯,整体可读性较好 Suggestions:
🔍 Diff Quality
Summary: The translation sync correctly modified the same three files as the English source, preserved section order and structure, and updated the translation heading-maps to reflect renamed/re-cased headings and titles. Issues:
This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
Pull request overview
This automated translation-sync PR updates the Chinese lecture content for the likelihood/Bayes materials to match upstream style-guide changes from QuantEcon/lecture-python.myst (PR #651), including code-cell style and RNG handling in simulations.
Changes:
- Adjusts lecture prose/formatting and MyST/JupyterBook markup (figure metadata, equation labels, spacing).
- Refactors likelihood-ratio simulation helpers to use an injected RNG (
rng) instead of globalnp.randomin the likelihood ratio process lectures. - Updates translation-sync state metadata (source SHA, sync date, mode, tool version).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lectures/likelihood_ratio_process.md | Style/format updates and RNG-injection refactor in likelihood-ratio simulation helpers. |
| lectures/likelihood_ratio_process_2.md | Style/format updates and RNG-injection refactor for the simulate helper. |
| lectures/likelihood_bayes.md | Style-guide alignment: headings normalization, MyST figure metadata, equation labeling, and formatting tweaks. |
| .translate/state/likelihood_ratio_process.md.yml | Updates translation-sync state (source SHA/date/mode/tool version). |
| .translate/state/likelihood_ratio_process_2.md.yml | Updates translation-sync state (source SHA/date/mode/tool version). |
| .translate/state/likelihood_bayes.md.yml | Updates translation-sync state (source SHA/date/mode/tool version). |
Suppressed comments (1)
lectures/likelihood_ratio_process.md:181
simulate_sequencesnow takesrngas the 4th positional argument, but the examples later in this lecture still call it assimulate_sequences('g', f, g, (F_a, F_b), (G_a, G_b), ...). With the current signature, the(F_a, F_b)tuple is bound torng, leading to an AttributeError whensimulate_likelihood_ratiosexecutesrng.beta(...). To keep existing calls working while still supporting an injected RNG, moverngto the end (optional) and initialize a default generator when it’s not provided.
def simulate_sequences(distribution, f_func, g_func, rng,
F_params=(1, 1), G_params=(3, 1.2), T=50, N=500):
"""
Generate N sequences of T observations from specified distribution.
"""
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Automated Translation Sync
This PR contains automated translations from QuantEcon/lecture-python.myst.
Source PR
#651 - [likelihood_bayes] Update lecture according to the style guide
Files Updated
lectures/likelihood_bayes.md.translate/state/likelihood_bayes.md.ymllectures/likelihood_ratio_process.md.translate/state/likelihood_ratio_process.md.ymllectures/likelihood_ratio_process_2.md.translate/state/likelihood_ratio_process_2.md.ymlDetails
This PR was created automatically by the translation action.