Skip to content

🌐 [translation-sync] [eig_circulant.md] Update np.random → Generator API - #222

Open
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-07-31T23-21-32-pr-994
Open

🌐 [translation-sync] [eig_circulant.md] Update np.random → Generator API#222
mmcky wants to merge 2 commits into
mainfrom
translation-sync-2026-07-31T23-21-32-pr-994

Conversation

@mmcky

@mmcky mmcky commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#994 - [eig_circulant.md] Update np.random → Generator API

Files Updated

  • ✏️ lectures/eig_circulant.md
  • ✏️ .translate/state/eig_circulant.md.yml

Details

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

This PR was created automatically by the translation action.

Copilot AI review requested due to automatic review settings July 31, 2026 23:21
@mmcky mmcky added the action-translation PRs created by QuantEcon/action-translation label Jul 31, 2026
Copilot AI review requested due to automatic review settings July 31, 2026 23:21
@mmcky mmcky added the automated Automated sync PR opened by action-translation label Jul 31, 2026
@netlify

netlify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploy Preview for astonishing-narwhal-a8fc64 ready!

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

@github-actions

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-07-31
Routing: auto-merge — no gating findings; floors met


📝 Translation Quality

Criterion Score
Accuracy 9/10
Fluency 9/10
Terminology 9/10
Formatting 10/10
Overall 9.2/10

Summary: The translation of the 'Associated Permutation Matrix' section is accurate, fluent, and faithfully preserves the mathematical content, code, and formatting of the English source. No syntax errors were found, and terminology is consistent with the glossary and rest of the document. This section does not present any notable issues requiring correction. The 'Associated Permutation Matrix' section is translated accurately and completely, preserving all mathematical notation, code blocks, and equation references without loss of meaning. Full-width Chinese punctuation is consistently used throughout the changed section, complying with the style policy. Technical phrasing such as '每个特征向量也是 $C$ 的特征向量' and '关联的特征值' is natural, precise, and consistent with terminology used elsewhere in the document.

Suggestions:

  • [minor · fluency] lectures/eig_circulant.md — ## 关联的置换矩阵 (heading translation): The section heading '## Associated Permutation Matrix' is translated as '## 关联的置换矩阵', which is accurate and consistent with the translation metadata mapping ('Associated Permutation Matrix': '关联的置换矩阵'). No issue found here; this is confirmed correct.
  • [minor · fluency] lectures/eig_circulant.md — 接下来,我们进行计算以验证方程 {eq}eqn:circulant 中定义的循环矩阵 $C$ 可以写成: The phrase 'execute calculations to verify' is translated as '进行计算以验证', which is accurate and natural. This is a well-handled sentence with no issues.

🔍 Diff Quality

Check Status
Scope Correct
Position Correct
Structure Preserved
Heading-map Correct
Overall 10/10

Summary: The code cell change replacing np.random.random with the modern Generator-based rng.random API was correctly and identically applied to the target document in the same position and section.


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

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.

🟢 Ready to approve

The changes are small, consistent with existing lecture patterns, and limited to a straightforward RNG API update plus translation-sync metadata refresh.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This automated translation-sync PR updates the eig_circulant lecture’s example code to use NumPy’s Generator API (default_rng) instead of the legacy global np.random API, and refreshes the translation state metadata to match the new upstream source commit.

Changes:

  • Updated the random-draw example in lectures/eig_circulant.md from np.random.random to rng = np.random.default_rng(); rng.random(...).
  • Updated .translate/state/eig_circulant.md.yml to reflect the new source SHA, sync date, mode, and tool version.
File summaries
File Description
lectures/eig_circulant.md Switches a lecture code-cell to the NumPy Generator random API.
.translate/state/eig_circulant.md.yml Updates translation-sync state to the latest upstream commit and tooling.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@github-actions

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request July 31, 2026 23:51 Inactive
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