Skip to content

[action-translation] resync: kalman.md - #118

Merged
mmcky merged 3 commits into
mainfrom
resync/kalman
Jul 19, 2026
Merged

[action-translation] resync: kalman.md#118
mmcky merged 3 commits into
mainfrom
resync/kalman

Conversation

@mmcky

@mmcky mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Forward Resync: kalman.md

Source: QuantEcon/lecture-python.mystlectures/kalman.md
Source commit: 7c6396a
This PR resyncs the translation to match the current source document.

Reason: The Chinese version contains substantial extra content not in the English source: a custom bivariate_normal function with docstring, additional figure references (kl_ex2_fig.png, kalman_ex3.png) not in source, extra explanatory text in exercise 2 and 3 (e.g. 'your figure should show errors decreasing irregularly', expected output description), and a footnote [^f1] citing Bishop2006 page 93 replacing the inline {note} block. It also omits some source content (e.g. detailed explanation of prior/measurement model context, the 'note' about x_hat notation, some paragraph text about President/Manhattan example, and the missile launched from 'hostile country' vs 'Y国'). Overall there are both additions unique to target and some content changes/omissions from source, but the presence of extra figures and expanded exercise text not present in source means TARGET_HAS_ADDITIONS takes priority.

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 07:42
@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 8a3d71e
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6a5c29613063800007e4ad31
😎 Deploy Preview https://deploy-preview-118--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 the Kalman filter lecture (lectures/kalman.md) to match the current upstream English source, and records the resync metadata under .translate/state/.

Changes:

  • Resynced lectures/kalman.md content, including updated notation, rewritten explanations, and updated code cells/plots to align with the upstream lecture.
  • Added translation frontmatter (title + heading mappings) for navigation/structure alignment.
  • Added resync state metadata file .translate/state/kalman.md.yml pointing to the upstream source commit SHA.

Reviewed changes

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

File Description
lectures/kalman.md Whole-document translation resync, including updated math notation, narrative, exercises, and code cell modernization.
.translate/state/kalman.md.yml Records the upstream source SHA, sync timestamp, model, and resync mode for this file.

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

Comment thread lectures/kalman.md
\hat x^F := \hat x + \Sigma G' (G \Sigma G' + R)^{-1}(y - G \hat x)
\quad \text{和} \quad
\Sigma^F := \Sigma - \Sigma G' (G \Sigma G' + R)^{-1} G \Sigma
\mu^F := \mu + \Sigma G^\top (G \Sigma G^\top + R)^{-1}(y - G \mu)
Comment thread lectures/kalman.md
\begin{aligned}
\hat x_{new} &:= A \hat x + K_{\Sigma} (y - G \hat x) \\
\Sigma_{new} &:= A \Sigma A' - K_{\Sigma} G \Sigma A' + Q \nonumber
\mu_{\mathrm{new}} &:= A \mu + K_{\Sigma} (y - G \mu) \\
@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
These {doc} targets exist only in lecture-python.myst until Phase 2
translates them; qualifying with the intermediate: intersphinx prefix
gives working links now, and a future resync restores local refs once
the targets exist. Program decision recorded 2026-07-18 (Matt).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 18, 2026

Copy link
Copy Markdown

✅ Translation Quality Review

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


📝 Translation Quality

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

Summary: This is a high-quality resync translation that accurately preserves the mathematical content, code, and structure of the original document. Terminology is consistent with standard Chinese academic conventions for Kalman filter concepts. The main concerns are the substitution of Wikipedia links with Zhihu article links for 'Woodbury matrix identity' and 'discrete-time algebraic Riccati equation', and the addition of an 'intermediate:' namespace prefix to the kalman_filter_var reference that doesn't appear in the English source - both worth verifying for intentionality. No markdown/MyST syntax errors were found. Mathematical notation, LaTeX equations, and code blocks are preserved accurately and completely throughout the resynced document. Technical terminology such as '先验分布' (prior), '滤波分布' (filtering distribution), '预测分布' (predictive distribution), and '卡尔曼增益' (Kalman gain) is consistent and matches standard Chinese statistical/econometric usage. The added YAML 'translation' metadata block with title and headings mapping is correctly implemented per the expected sync system format. Complex mathematical explanations (e.g., Bayes' theorem, Riccati equations, convergence conditions) are translated with accurate technical meaning and appropriate academic register.

Suggestions:

  • Link substitution: The English source links 'Woodbury matrix identity' to Wikipedia, but the Chinese translation replaces it with a Zhihu article link (https://zhuanlan.zhihu.com/p/388027547). Similarly 'discrete-time algebraic Riccati equation' links to Wikipedia in English but to a Zhihu article in Chinese. These substitutions change the reference source and should ideally preserve the original Wikipedia links or note this as an intentional localization choice.
  • Reference to kalman_filter_var: The English source uses '{doc}kalman_filter_var' without a namespace prefix, but the Chinese translation uses '{doc}intermediate:kalman_filter_var'. Verify this added namespace prefix is correct and intentional for the Chinese docs structure, as it doesn't match the source's simpler reference.
  • Minor phrasing: '除了Anaconda中已有的库外' is accurate but slightly less fluent than possible alternatives like '除了Anaconda中包含的库外' — this is a very minor stylistic note and not a significant issue.

🔍 Diff Quality

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

Summary: The target document was correctly resynced to match the current English source structure, content, and code cells, with an appropriate translation heading map added.


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

@github-actions
github-actions Bot temporarily deployed to pull request July 18, 2026 11:34 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request July 19, 2026 01:58 Inactive
@mmcky
mmcky merged commit 64852a9 into main Jul 19, 2026
7 checks passed
@mmcky
mmcky deleted the resync/kalman branch July 19, 2026 02: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