🌐 [translation-sync] [organization_capital.md] Update np.random → Generator API - #213
🌐 [translation-sync] [organization_capital.md] Update np.random → Generator API#213mmcky wants to merge 2 commits into
Conversation
✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Translation Quality ReviewVerdict: PASS | Model: claude-sonnet-5 | Date: 2026-07-31 📝 Translation Quality
Summary: The translation of the Bayesian learning section is accurate, fluent, and terminologically consistent. Mathematical formulas and equation labels are preserved intact, and the technical vocabulary aligns well with standard econometrics/statistics terminology in Chinese. Only very minor stylistic nitpicks were found; no substantive accuracy or formatting issues exist in this section. The Bayesian learning section accurately preserves all mathematical notation and equations without alteration Technical terms like 后验分布、后验均值、后验精度 are used consistently and correctly throughout the section The explanation of the signal model and posterior updating flows naturally in Chinese academic register Suggestions:
🔍 Diff Quality
Summary: The core code-cell translation fix (np.random.seed to default_rng) was correctly applied, but the target document also removed blank lines before many section headings that were not changed in the source, introducing unintended structural deviations. Issues:
This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
Pull request overview
This automated translation-sync PR updates the Chinese organization_capital lecture to align with the upstream change that migrates legacy np.random usage to the NumPy Generator API, and refreshes the translation sync state metadata accordingly.
Changes:
- Replaced
np.random.seed(...)/np.random.randn(...)usage withrng = np.random.default_rng(...)andrng.standard_normal(...)in an updated lecture code cell. - Removed several whitespace-only blank lines in the markdown to normalize formatting.
- Updated
.translate/state/organization_capital.md.ymlmetadata (source SHA, sync timestamp, model, mode, tool version).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/organization_capital.md | Updates the lecture’s random number generation example to use the Generator API and cleans up whitespace-only blank lines. |
| .translate/state/organization_capital.md.yml | Updates translation-sync state metadata to reflect the new upstream source commit and tooling/model used. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Automated Translation Sync
This PR contains automated translations from QuantEcon/lecture-python.myst.
Source PR
#1014 - [organization_capital.md] Update np.random → Generator API
Files Updated
lectures/organization_capital.md.translate/state/organization_capital.md.ymlDetails
This PR was created automatically by the translation action.