Repository navigation
[sir_model] update code and text - #1050
Merged
Merged
Conversation
longye-tian
added
enhancement
23 tasks done
📖 Netlify Preview Ready!Preview URL: https://pr-1050--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
longye-tian
marked this pull request as ready for review
September 6, 2026 07:10
longye-tian
removed
the
do-not-merge
Contributor
|
Thanks @longye-tian ! Much appreciated. All changes are good. Merging. |
4 tasks done
Contributor
|
\translate-resync Re-running the zh-cn sync of |
Contributor
✅ Translation sync completed (zh-cn)Target repo: QuantEcon/lecture-python.zh-cn
|
mmcky
added a commit
to QuantEcon/lecture-python.zh-cn
that referenced
this pull request
Oct 22, 2026
The 2026-08-20 quality audit's one major finding on this lecture: the Chinese said to delay the infection peak until *before* a vaccine is found, where the source's point is the reverse, that pushing the peak further into the future may reduce cumulative deaths if a vaccine is found. QuantEcon/lecture-python.myst#1050 only touched this sentence's wording ("the peak of the curve"), so the resync kept the existing translation. Retranslated to follow the source. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mmcky
added a commit
to QuantEcon/lecture-python.zh-cn
that referenced
this pull request
Oct 22, 2026
* Update translation: lectures/sir_model.md * Update translation: .translate/state/sir_model.md.yml * sir_model: correct the inverted closing sentence The 2026-08-20 quality audit's one major finding on this lecture: the Chinese said to delay the infection peak until *before* a vaccine is found, where the source's point is the reverse, that pushing the peak further into the future may reduce cumulative deaths if a vaccine is found. QuantEcon/lecture-python.myst#1050 only touched this sentence's wording ("the peak of the curve"), so the resync kept the existing translation. Retranslated to follow the source. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * sir_model: drop a figure.figsize line the source does not have plt.rcParams["figure.figsize"] = (11, 5) sat in the imports cell with no counterpart in the English, one of eight such lines the 2026-08-20 audit found riding along with the font configuration. Removed, so the cell's code matches the source apart from the i18n font block. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * sir_model: space before three links and roles The zh-cn rule (QuantEcon/project-translation#70): a space before an inline role or link. Three lines in passages this resync did not change still ran them into the preceding character. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
mmcky
added a commit
to QuantEcon/lecture-python.zh-cn
that referenced
this pull request
Oct 22, 2026
#302, the sync of QuantEcon/lecture-python.myst#1050, carried every English change, but it also rewrote passages whose English did not change. This puts the reviewed text back there (the 2025 review by yuki and Humphrey Yang, and the March 2025 text they kept) and keeps every change the English made. Text and plot labels only: structural parity holds, the 25 code cells match the source with strings masked, and both math blocks are identical. - 时间路径: nine reviewed lines, among them 主要关注的是 and its two bullets (yuki's), and the partition sentence, whose R is math again. - 参数: its four reviewed lines, so 阿特克森 is Atkeson again, as in the rest of the lecture. - The state list reads 易感者…移出者 again, as the intro does, with the English's new $S$, $E$, $I$, $R$. - Plot labels are Chinese throughout: 人口比例 on the four y-axes that were English (解除封锁 already used it) and 天数 on both x-axes. Lines that the July forward resync (#151) wrote and #302 reworded keep #302's text. Both zh-cn rules hold on every prose line (QuantEcon/project-translation#70). Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi @jstac
This PR updates the
sir_modellecture according to #1049Best,
Longye