Skip to content

🌐 [translation-sync] [numpy] Remove the duplicated {index} directive under Matrix Multiplication - #175

Merged
mmcky merged 2 commits into
mainfrom
translation-sync-2026-11-02T00-33-44-pr-632
Nov 2, 2026
Merged

mmcky merged 2 commits into
mainfrom
translation-sync-2026-11-02T00-33-44-pr-632

Conversation

@mmcky

@mmcky mmcky commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Automated Translation Sync

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

Source PR

#632 - [numpy] Remove the duplicated {index} directive under Matrix Multiplication

Files Updated

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

Details

  • Source Language: en
  • Target Language: fa
  • Model: claude-sonnet-5

This PR was created automatically by the translation action.

@mmcky mmcky added action-translation PRs created by QuantEcon/action-translation automated Automated sync PR opened by action-translation labels Nov 2, 2026
Copilot AI balanced review requested due to automatic review settings November 2, 2026 00:33
@netlify

netlify Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for majestic-griffin-10b166 ready!

Name Link
🔨 Latest commit 3c3b26a
🔍 Latest deploy log https://app.netlify.com/projects/majestic-griffin-10b166/deploys/6ae7da6fd64b30ae903d561f
😎 Deploy Preview https://deploy-preview-175--majestic-griffin-10b166.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

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

✅ Translation Quality Review

Verdict: PASS | Model: claude-sonnet-5 | Date: 2026-11-02
Routing: auto-merge — no gating findings; floors met
Shadow gate: would auto-merge (recorded only; no action taken)


📝 Translation Quality

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

Summary: The translated Matrix Multiplication section is accurate, fluent, and well-formatted, with correct terminology and intact MyST syntax. Only a very minor stylistic nit was found regarding a slightly literal phrase. The Matrix Multiplication section accurately preserves the meaning and code blocks of the English source. Terminology such as 'ضرب ماتریسی' and 'نماد @' is used consistently and correctly per the glossary. Markdown/MyST structure (headings, code-cell blocks, cross-reference labels) is fully preserved without syntax errors.

Suggestions:

  • [nit · fluency] lectures/numpy.md — ## ضرب ماتریسی: The phrase 'نماد @' is accurate, but the following sentence 'NumPy حدس آموزش‌دیده‌ای از آنچه شما می‌خواهید می‌زند' is a slightly literal rendering of 'makes an educated guess' and reads a bit awkwardly in Persian. → NumPy حدس هوشمندانه‌ای از منظور شما می‌زند

🔍 Diff Quality

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

Summary: The duplicate '{index} single: NumPy; Matrix Multiplication' block was correctly removed from the target document in the same location as the source, with no other unintended changes.


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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The review surfaced no unresolved concerns or outstanding issues with the straightforward translation sync and metadata update.

No open issues 🎉

What changed in this PR

This PR is an automated translation sync for the Farsi (fa) version of the QuantEcon numpy lecture. It mirrors upstream source PR #632, which removed a duplicated {index} directive under the "Matrix Multiplication" section. The change keeps the lecture content in parity with the English source and updates the translation state metadata accordingly.

Changes:

  • Removed the duplicated {index} single: NumPy; Matrix Multiplication directive block in lectures/numpy.md, leaving a single directive under the "ضرب ماتریسی" (Matrix Multiplication) heading.
  • Updated .translate/state/numpy.md.yml with the new source commit SHA, sync date (2026-11-02), and tool version (0.30.0).
File Description
lectures/​numpy.md Removes the redundant second {index} directive so only one index entry remains for the section.
.translate/​state/​numpy.md.yml Updates translation state metadata (source SHA, sync date, tool version) to reflect this sync.

I reviewed the diff and the surrounding content in lectures/numpy.md; the duplicate directive removal is correct, a single {index} directive remains under the Matrix Multiplication heading, and the state file update is consistent with the sync metadata. No issues found.

Review effort: Balanced

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request November 2, 2026 00:38 Inactive
@mmcky
mmcky merged commit e393384 into main Nov 2, 2026
10 checks passed
@mmcky
mmcky deleted the translation-sync-2026-11-02T00-33-44-pr-632 branch November 2, 2026 01:21

This branch was previously deployed

1 inactive deployment
pull request — 3c3b26a1 Deployed Nov 2, 2026 by github-actions[bot]
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