Glossary: use 边缘分布 for "marginal distribution" in inventory_dynamics#200
Merged
Conversation
The AI review on #198 flagged this and the PR merged before it was fixed, so the term is currently on main. The glossary is explicit and context-aware: "Marginal distribution" maps to 边缘分布 tagged `"context": "probability"`, while every economics sense correctly keeps 边际 — 边际产量, 边际消费倾向, 边际收入, 边际效用. This lecture uses the probability sense throughout: ψ_T is the distribution of X_T at a fixed time T, and the later reference is to the sequence {ψ_t} converging to a limiting distribution. 边缘分布 is the standard mathematical term for that; 边际 is "marginal" as in marginal cost. Four occurrences: the H2 heading, its heading-map value, and two body references. The heading-map value is updated alongside the heading deliberately — leaving them out of step would make the next sync mis-map the section.
✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR corrects the Chinese terminology for “marginal distribution” in the probability/statistics sense within the inventory_dynamics lecture, changing it from 边际分布 (economics “marginal”) to the standard mathematical term 边缘分布. This aligns the section heading, the translation heading-map, and in-text references so future sync tooling maps the section correctly.
Changes:
- Update the front-matter
translation.headingsentry for “Marginal distributions” to 边缘分布 - Rename the corresponding H2 heading to 边缘分布
- Update two body references to use 边缘分布 consistently
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Fixes the terminology defect the AI review caught on #198, which merged before it was addressed and is currently on
main.The change
Four occurrences of 边际分布 → 边缘分布 in
inventory_dynamics.md: the H2 heading, its heading-map value, and two body references.The glossary is explicit and context-aware here —$\psi_T$ is the distribution of $X_T$ at a fixed time $T$ , and the later reference is to the sequence ${\psi_t}$ converging to a limiting distribution. 边缘分布 is the standard mathematical term for that; 边际 is "marginal" as in marginal cost.
"Marginal distribution"maps to 边缘分布 with"context": "probability", while every economics sense correctly keeps 边际 (边际产量, 边际消费倾向, 边际收入, 边际效用). This lecture is the probability sense throughout:The heading-map value is updated alongside the heading deliberately. Leaving them out of step would make the next sync mis-map the section.
The same defect exists in eight other lectures
Found while checking the scope, and deliberately not fixed here — each needs its own read, and bundling fifty-odd edits across nine unrelated lectures into one PR would make it unreviewable.
prob_matrix.mdfinite_markov.mdak_aiyagari.mdaiyagari.mdar1_turningpts.mdlake_model.mdmle.mdmulti_hyper.mdSampled contexts are all the probability sense — "discrete marginal distribution", "sample marginal distribution", a$a_t$ " — so all appear to be the same defect rather than deliberate usage. Worth confirming lecture by lecture before changing them.
Marginal Distributionsheading, "the marginal distribution ofThe edition is currently inconsistent with itself: six other lectures already use 边缘分布 correctly.
lecture-intro.zh-cnhas one affected file too. Happy to follow up with a sweep if wanted.Why this one is worth the paperwork
Beyond the term itself, this is the first genuine catch by reviewer verdict v2 in production, and it merged unactioned — which is a useful signal for the human-review program (QuantEcon/project-translation#15). The
editorrouting worked exactly as designed; what is missing is an owner, which is what Stage 2 routing adds. Recorded on QuantEcon/action-translation#147.