Skip to content

🌐 [translation-sync] Add a lecture on Social Security trust fund depletion - #296

Open
mmcky wants to merge 1 commit into
mainfrom
translation-sync-2026-10-16T20-24-18-pr-1091
Open

mmcky wants to merge 1 commit into
mainfrom
translation-sync-2026-10-16T20-24-18-pr-1091

Conversation

@mmcky

@mmcky mmcky commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

This PR contains automated translations from QuantEcon/lecture-python.myst.

Source PR

#1091 - Add a lecture on Social Security trust fund depletion

Files Updated

  • ✏️ lectures/_toc.yml

⚠️ Files Failed to Translate

The following new file(s) could not be translated and are absent from this PR. Their entries have been removed from lectures/_toc.yml so the target build stays green; the entries return with the files once they are delivered. The error details and recovery steps are in the failure issue this run opens on the source repository.

  • ❌ lectures/social_security_depletion.md

Details

  • Source Language: en
  • Target Language: zh-cn
  • Model: claude-sonnet-5

This PR was created automatically by the translation action.

@mmcky mmcky added the action-translation PRs created by QuantEcon/action-translation label Oct 16, 2026
Copilot AI balanced review requested due to automatic review settings October 16, 2026 20:24
@mmcky mmcky added the automated Automated sync PR opened by action-translation label Oct 16, 2026
@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!

Name Link
🔨 Latest commit 34987e4
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6ad287f7fdc53a3f97bc6bb7
😎 Deploy Preview https://deploy-preview-296--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.

🟡 Changes recommended

The added _toc.yml entries reference four lecture files that do not exist in the repository, which will break the jb build lectures -W CI build.

1 open finding
What changed in this PR

This is an automated translation-sync PR from QuantEcon/lecture-python.myst (source PR #1091). Its stated purpose is to add the "Social Security trust fund depletion" lecture to the Chinese translation book. However, the lecture itself (social_security_depletion.md) failed to translate and is absent, so the only file actually changed is lectures/_toc.yml.

The diff does not match the stated intent: instead of handling social_security_depletion, it adds table-of-contents entries for four unrelated lectures (gallant_tauchen, lucas_prescott_investment, optimal_growth_uncertainty, lq_mean_field_games) whose .md files do not exist in this repository. Since CI builds with jb build lectures -n -W, these ToC references to missing documents will break the build.

Changes:

  • Adds four - file: ToC entries referencing lecture files that are not present in lectures/ (build-breaking).
  • Introduces an incidental trailing-whitespace change on the 统计与信息论 caption line (harmless).
  • Does not add or reference social_security_depletion, contrary to the PR description.
File Description
lectures/​_toc.yml Adds ToC entries for four lectures whose source files are missing from the repo, which will fail the Jupyter Book build; does not touch the social_security_depletion lecture named in the PR.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lectures/_toc.yml
Comment on lines 52 to +53
- file: market_diffusion
- file: gallant_tauchen

This branch has not been deployed

No deployments
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 automated Automated sync PR opened by action-translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants