🌐 [translation-sync] Link the probability sequence together - #283
Conversation
mmcky
added
action-translation
✅ Deploy Preview for playful-platypus-17d3bb ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
| Criterion | Score |
|---|---|
| Accuracy | 9/10 |
| Fluency | 9/10 |
| Terminology | 8/10 |
| Formatting | 9/10 |
| Overall | 8.8/10 |
Summary: 翻译整体质量很高，四个被修改的章节（最大似然估计、概述、与概率分布的联系）准确传达了原文含义，专业术语翻译规范，格式和MyST语法保持完整。仅存在少量可选性改进点，如外部链接来源从维基百科改为百度百科（不影响准确性）以及个别术语链接的细微差异，无重大问题。 专业术语（如最大似然估计、似然函数、对数似然函数、帕累托分布等）翻译准确且与术语表保持一致 数学公式和代码块完整保留，格式正确 长句拆分自然，读起来符合学术中文的表达习惯 章节标题的翻译准确传达了原意（如“到底什么才是最好的分布？”生动且贴切）
Suggestions:
- [minor · terminology] lectures/mle.md — ## 最大似然估计: 最大似然估计和似然函数的链接被改为百度百科链接（如 https://baike.baidu.com/item/最大似然估计/4967925），而原文使用维基百科链接。虽然这不影响译文准确性，但与原文的引用来源不一致，且百度百科条目命名可能与术语略有差异（如'似然函数'条目链接文字为'似然函数/6011241'）。 → 如无特殊政策要求替换为百度百科，建议保留与英文原文一致的维基百科链接以保持引用来源的一致性。
- [minor · accuracy] lectures/mle.md — ## 最大似然估计, PDF公式: 译文修正了原文公式中的一个错误（原文exp部分的右括号位置有误，写成 \exp(...)\right)^2 而不是 \exp(...^2)），这虽然使数学上更正确，但已经偏离了'MODIFIED'范围要求仅评估翻译质量而非纠正原文错误；此改动值得在评审中标注但不算翻译缺陷。 → 无需修改，此为对原文公式错误的合理修正，建议在PR说明中注明此为公式修正而非翻译误差。
- [minor · terminology] lectures/observed_distributions.md — ## 概述 (Outline): 英文原文 Outline 部分中提到 '{doc}
fitting_distributions' 时并未提供中文译名（如“拟合分布”），而是直接使用英文doc链接文字，纵观全篇其余{doc}引用也未标注中文标题，是否需要为一致性统一处理有待确认，但这不算严重问题。 → 可考虑为 {doc}fitting_distributions添加简短中文说明，例如 '{doc}拟合分布 <fitting_distributions>'，以提高可读性，但非必须。
🔍 Diff Quality
| Check | Status |
|---|---|
| Scope Correct | ✅ |
| Position Correct | ❌ |
| Structure Preserved | ❌ |
| Heading-map Correct | ❌ |
| Overall | 2.5/10 |
Summary: While mle.md and prob_dist.md show correctly scoped, positioned, and structurally sound syncs, observed_distributions.md contains extensive unrelated changes (stripped figure metadata, reworded prose, altered heading translations) far beyond the two sentences added in the source.
Issues:
- observed_distributions.md has +254/-136 line changes, vastly exceeding the two sentences added in the source document, indicating unrelated content was altered
- Numerous mystnb figure directives (caption/name metadata) were stripped from code cells throughout observed_distributions.md target, which is a structural regression not present in the source diff
- The 'Outline' heading translation changed from '概览' to '概述' in observed_distributions.md frontmatter without any corresponding change in the source heading text
- Large sections of previously-existing translated prose were reworded/retranslated even though the corresponding English source sentences were unchanged, going beyond simple sync of new content
- The scale of unrelated changes suggests the translation sync introduced drift or was based on a different/newer translation baseline rather than a minimal sync of the two new sentences
This review was generated automatically by action-translation review mode.
There was a problem hiding this comment.
🟡 Not ready to approve
There are broken {doc} references to a missing fitting_distributions page and a couple of plotting code issues that can cause build/runtime failures.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This automated translation-sync PR brings in the new “观测分布 (observed distributions)” lecture from the upstream English source and updates the existing MLE lecture, along with translation state metadata files.
Changes:
- Added a new lecture
lectures/observed_distributions.md(Chinese) covering sample moments/quantiles and visualization tools (histograms, ECDF, KDE, box/violin plots). - Updated
lectures/mle.mdwith a new cross-reference to the distribution-fitting lecture. - Added/updated
.translate/state/*.ymlfiles to record sync metadata (source SHA, date, tool version).
File summaries
| File | Description |
|---|---|
| lectures/observed_distributions.md | New translated lecture content, including multiple executable plotting examples and cross-references. |
| lectures/mle.md | Adds a new explanatory sentence linking MLE workflow to the distribution-fitting lecture. |
| .translate/state/observed_distributions.md.yml | Records sync metadata for the newly added lecture. |
| .translate/state/mle.md.yml | Updates sync metadata for the MLE lecture to the new source SHA/date/tool version. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 6
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
|
|
||
| 我们将讨论如何总结和可视化这些数据，以及观测数据如何与概率分布联系起来。 | ||
|
|
||
| 第三讲 {doc}`fitting_distributions` 接着讨论哪个概率分布最适合描述给定的数据集这一问题。 |
|
|
||
| 在这里，我们通过将样本均值设为正态分布的均值，并将样本方差设为正态分布的方差，将正态分布与亚马逊月度回报率相匹配。 | ||
|
|
||
| （这个方法被称为矩方法，{doc}`fitting_distributions` 将对其进行完整的阐述。） |
| 1. 猜测潜在分布是什么（例如，均值为 $\mu$，标准差为 $\sigma$ 的正态分布）。 | ||
| 2. 估计参数值（例如，估计正态分布的 $\mu$ 和 $\sigma$）。 | ||
|
|
||
| 这与 {doc}`fitting_distributions` 中讨论的两个步骤是一样的，不过那里是用矩方法来选择参数，并且还展示了如何判断拟合结果的好坏。 |
| name: fig:amazon-hist | ||
| --- | ||
| fig, ax = plt.subplots() | ||
| ax.hist(x_amazon, bins=20) |
| fig, ax = plt.subplots() | ||
| ax.violinplot(x_amazon) | ||
| ax.set_ylabel('monthly return (percent change)') | ||
| ax.set_xlabel('KDE') |
| name: fig:retailer-violin | ||
| --- | ||
| fig, ax = plt.subplots() | ||
| ax.violinplot([x_amazon['AMZN'], x_costco['COST']]) |
|
Error: translation produced 1 error(s); branch left untouched: Error processing lectures/prob_dist.md: Error: structural parity check failed for lectures/prob_dist.md:
You may need to manually resolve conflicts or run |
The sync for QuantEcon/lecture-python-intro#819 declared lectures/prob_dist.md in its metadata but emitted no diff for it, the same dropped-file fault seen in #278 and #280. Upstream changed that file +7 -2, rewriting the outline to introduce the three-lecture sequence. Added the three sequence paragraphs to prob_dist.md, translated to match the register this PR already uses for the same cross-links in observed_distributions.md. Both {doc} targets are on main, so they resolve. Upstream's edit replaces an earlier "A companion lecture" paragraph, but that paragraph never reached this repo — it belongs to the wider prob_dist.md gap from QuantEcon/lecture-python-intro#811, tracked in #290 — so the net change here is an addition. prob_dist's state file is deliberately left untouched so that gap stays flagged for resync. Also rebased onto current main. observed_distributions.md and its state file conflicted add/add against the copy #278 landed; both resolved in favour of this branch, whose version is the strict superset — 903 lines against 785, with 19 figure captions and both fitting_distributions cross-links that main's older copy lacks. Verified against upstream at a453cf5: same 2 anchors, 13 headings, 42 code cells and 19 caption blocks, with no duplicate labels or figure names repo-wide. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ff085b7 to
643d918
Compare
Automated Translation Sync
This PR contains automated translations from QuantEcon/lecture-python-intro.
Source PR
#819 - Link the probability sequence together
Files Added
lectures/observed_distributions.md.translate/state/observed_distributions.md.ymlFiles Updated
lectures/mle.md.translate/state/mle.md.ymlDetails
This PR was created automatically by the translation action.