Skip to content

[action-translation] resync: mccall_model.md#133

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

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

Conversation

@mmcky

@mmcky mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Forward Resync: mccall_model.md

Source: QuantEcon/lecture-python.mystlectures/mccall_model.md
Source commit: 0bfcac8
This PR resyncs the translation to match the current source document.

Reason: The target is based on an entirely older version of this lecture (using Numba/jitclass instead of JAX, different code structure, older exercise sets). It contains substantial content not in source: full 'Exercise 2' with continuous distribution solution using jitclass, additional explanatory text, different exercise parameters, etc. Meanwhile it's missing major sections present in source: the entire 'Volatility' section (mean-preserving spread, lifetime value and volatility analysis), the JAX-based implementation details, comparative statics with jitted functions, and updated GPU admonition include. Since target has unique content not in source, TARGET_HAS_ADDITIONS takes priority, though there are also many CONTENT_CHANGES (missing volatility section, different library/approach, different exercise solutions).

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:17
@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 506d184
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6a5c1318a6d39a0008bea068
😎 Deploy Preview https://deploy-preview-133--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 mccall_model.md to match the current upstream lecture, including the newer JAX-based implementation and the added “Volatility” section, and records the sync state metadata.

Changes:

  • Resynced lectures/mccall_model.md to the upstream structure/content (JAX-based code path, updated sections/exercises, GPU admonition, volatility analysis).
  • Added translation sync state file .translate/state/mccall_model.md.yml with source SHA and sync metadata.

Reviewed changes

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

File Description
lectures/mccall_model.md Whole-file resync of the translated lecture content and embedded code cells to the updated upstream version (JAX + new sections).
.translate/state/mccall_model.md.yml Records resync metadata (source SHA, sync date, mode, model/tool versions).

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

Comment thread lectures/mccall_model.md
Comment on lines +445 to +454
c, β, w, q = model
i = 0
error = tol + 1
v = v_init

while i < max_iter and error > tol:
v_next = T(model, v)
error = jnp.max(jnp.abs(v_next - v))
v = v_next
i += 1
Comment thread lectures/mccall_model.md
为了证明这一点,我们将:

1. 计算每个波动性水平下的保留工资
3. 利用蒙特卡洛方法,计算与该保留工资相关的终身收入流的预期折现价值。
Comment thread lectures/mccall_model.md
Comment on lines 80 to +84
import matplotlib.pyplot as plt
import matplotlib as mpl
FONTPATH = "fonts/SourceHanSerifSC-SemiBold.otf"
mpl.font_manager.fontManager.addfont(FONTPATH)
plt.rcParams['font.family'] = ['Source Han Serif SC']

plt.rcParams["figure.figsize"] = (11, 5) #设置默认图片尺寸
import numpy as np
from numba import jit, float64
from numba.experimental import jitclass
import numba
import jax
import jax.numpy as jnp
@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
The resynced file keeps Chinese plot labels but reverted the import
cell to the source's exact form, losing the Source Han Serif setup -
Chinese in figures would render as missing glyphs. Residual #107
class found in the merge review; applied wave-wide.

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-18


📝 Translation Quality

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

Summary: 整体翻译质量优秀,准确传达了原文的经济学和数学概念,术语使用规范一致,代码块和公式格式保留完好。发现的问题主要是细节性的(如链接来源、格式细微差异),不影响整体理解和技术准确性。 数学公式和LaTeX代码块完整保留,编号引用(如{eq}odu_pv)正确无误 专业术语翻译准确且一致,如'贝尔曼方程'、'保留工资'、'折现因子'等均符合经济学/动态规划领域惯例 代码块中的注释都被恰当地翻译成中文,同时保留了变量名和函数结构不变,符合Jupyter notebook代码规范 长难句的处理流畅自然,如关于Trade-Off和Fixed Point Theory部分的多步骤逻辑推导清晰易懂

Suggestions:


🔍 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, with all new sections, code changes (Numba to JAX), removed exercise, and heading metadata properly translated and positioned.


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

@github-actions
github-actions Bot temporarily deployed to pull request July 18, 2026 11:33 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request July 19, 2026 00:14 Inactive
@mmcky
mmcky merged commit 11783bf into main Jul 19, 2026
7 checks passed
@mmcky
mmcky deleted the resync/mccall_model 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