Skip to content

[action-translation] resync: prob_matrix.md#146

Merged
mmcky merged 2 commits into
mainfrom
resync/prob_matrix
Jul 19, 2026
Merged

[action-translation] resync: prob_matrix.md#146
mmcky merged 2 commits into
mainfrom
resync/prob_matrix

Conversation

@mmcky

@mmcky mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Forward Resync: prob_matrix.md

Source: QuantEcon/lecture-python.mystlectures/prob_matrix.md
Source commit: 78030a3
This PR resyncs the translation to match the current source document.

Reason: This is a substantially different/older version of the lecture. The target lacks major newer sections from source (probability space/measure formal definitions details, frequentist vs Bayesian subsection details, transition probability matrix formatted differently, Gaussian copula example, all 5 exercises with solutions, updated library imports like scipy.stats/comb/rng). Conversely, target contains substantial content not in source: a full narrative example about job interview daily wage with discrete-continuous variable, sample mean/variance code and analytical calculations for that example, and different phrasing/structure in several sections (e.g., probability space described narratively rather than with rigorous sigma-algebra definitions). Since target has unique examples/sections absent from source, this is TARGET_HAS_ADDITIONS.

Changes

Whole-file resync applied. The entire document was resynced in a single pass.


Created by action-translation forward resync

Copilot AI review requested due to automatic review settings July 18, 2026 08:44
@mmcky mmcky added action-translation-sync CLI resync PR (translate forward --github) resync Forward resync labels Jul 18, 2026
@netlify

netlify Bot commented Jul 18, 2026

Copy link
Copy Markdown

Deploy Preview for astonishing-narwhal-a8fc64 ready!

Name Link
🔨 Latest commit 925ae02
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6a5c131a91e9260008127a88
😎 Deploy Preview https://deploy-preview-146--astonishing-narwhal-a8fc64.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR forward-resyncs the Chinese translation of prob_matrix.md to align with the upstream English lecture, updating both the lecture content and the translation sync state metadata.

Changes:

  • Added translation frontmatter mappings and refreshed multiple sections to match the current upstream structure/content.
  • Updated code cells and imports (e.g., scipy, default_rng) and introduced new material (Gaussian copula example, exercises + solutions).
  • Added a translation sync state file recording source SHA and resync metadata.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
lectures/prob_matrix.md Whole-lecture resync: adds translation frontmatter, updates probability theory narrative, modernizes code cells, and adds new examples/exercises.
.translate/state/prob_matrix.md.yml Records resync provenance (source SHA/date/model/mode/section count).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lectures/prob_matrix.md
* $X$ 的可能值的数量是有限的或可数无限的
* 我们用**概率质量函数**(一个非负且和为1的序列)代替**密度**
* 在类似 {eq}`eq:CDFfromdensity` 这样的关联累积分布函数和概率质量函数的公式中,我们用求和代替积分
* 在密度存在的情况下,我们用求和代替像 {eq}`eq:CDFfromdensity` 这样公式中的积分
Comment thread lectures/prob_matrix.md
Comment on lines +503 to +504
\textrm{Prob}\{Y=j\vert X=i\} = \frac{\rho_{ij}}{ \sum_{j}\rho_{ij}}
= \frac{\textrm{Prob}\{Y=j, X=i\}}{\textrm{Prob}\{ X=i\}}
Comment thread lectures/prob_matrix.md

$$
p_{ij}=\textrm{Prob}\{Y=j \mid X=i\}=\frac{\rho_{ij}}{\sum_j \rho_{ij}},
p_{ij}=\textrm{Prob}\{Y=j|X=i\}= \frac{\rho_{ij}}{ \sum_{j}\rho_{ij}}
@mmcky mmcky added the action-translation PRs created by QuantEcon/action-translation label Jul 18, 2026
@github-actions

github-actions Bot commented Jul 18, 2026

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request July 18, 2026 10:30 Inactive
@github-actions

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-07-18


📝 Translation Quality

Criterion Score
Accuracy 9/10
Fluency 9/10
Terminology 9/10
Formatting 8/10
Overall 8.9/10

Summary: This is a high-quality, comprehensive resync translation that accurately conveys the mathematical and conceptual content of the English source while maintaining natural Simplified Chinese academic prose. Terminology is consistent with the glossary, and all code/math blocks are intact. Minor issues include inconsistent PrettyTable field-name translations across similar examples (e.g., coupling section table headers) and an unexplained addition of font-configuration code in the imports block, but these do not affect overall comprehension or correctness. Mathematical notation, LaTeX equations, and code blocks are fully preserved and correctly rendered throughout the resynced document. Technical terminology (联合分布, 边缘分布, 条件分布, 耦合, Copula函数, 充分统计量, etc.) is applied consistently and matches standard academic Chinese usage. Complex conceptual passages (e.g., probability space definitions, Bayes' law note, copula construction steps) are translated with high fidelity and natural academic register. Section headings and structure align well with the source, and the added YAML translation metadata follows expected conventions.

Suggestions:

  • Imports section: The translation adds extra font-configuration code (matplotlib font imports/settings) not present in the English source's import block. While this may be a project convention for CJK rendering, it constitutes an unrequested addition to a 'changed' section and should be verified against project policy.
  • 'Application: forecasting a time series' section: 'This formula is a workhorse for applied economic forecasters.' → translated as '这个公式是应用经济学预测者的常用工具' — accurate, but 'workhorse' connotes a heavily-relied-upon practical tool; consider '常用且实用的工具' or keeping as is since meaning is preserved adequately.
  • Gaussian copula section: 'Changing ρ controls the strength of dependence while the marginals remain unchanged.' is well translated, but note table headers use inconsistent naming style: some tables use '值/概率' while others use 'x值/x概率' — this is a minor consistency issue for prettytable field names introduced in this resync (e.g., 'c1_x_值' vs 'c1_x值' vs 'c2_x_取值'), suggest standardizing field name translations across all coupling examples for consistency.

🔍 Diff Quality

Check Status
Scope Correct
Position Correct
Structure Preserved
Heading-map Correct
Overall 10/10

Summary: The resync correctly restructures the target document to match the source's section order, adds the translation metadata block with accurate heading mappings, and removes stale/duplicated content, though the final target is missing translated solutions for exercises 3-5 that exist in the source.

Issues:

  • Some exercise solutions (ex3, ex4, ex5) present in source are missing from the target's 'After' document, though this appears to be a pre-existing gap in translation coverage rather than a resync error, since the diff generally tracks the source structure well
  • The target retains legacy content (e.g., the day-wage discrete-continuous example, some duplicated 'Transition probability matrix' text) removed - actually verified these were properly removed in After version, consistent with source

This review was generated automatically by action-translation review mode.

@github-actions
github-actions Bot temporarily deployed to pull request July 19, 2026 00:14 Inactive
@mmcky
mmcky merged commit 00c2fac into main Jul 19, 2026
7 checks passed
@mmcky
mmcky deleted the resync/prob_matrix branch July 19, 2026 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action-translation PRs created by QuantEcon/action-translation action-translation-sync CLI resync PR (translate forward --github) resync Forward resync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants