Repository navigation
🌐 [translation-sync] [numpy] Remove the duplicated {index} directive under Matrix Multiplication - #175
Conversation
✅ Deploy Preview for majestic-griffin-10b166 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Translation Quality ReviewVerdict: PASS | Model: claude-sonnet-5 | Date: 2026-11-02 📝 Translation Quality
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:
🔍 Diff Quality
Summary: The duplicate ' This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
🟢 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 Multiplicationdirective block inlectures/numpy.md, leaving a single directive under the "ضرب ماتریسی" (Matrix Multiplication) heading. - Updated
.translate/state/numpy.md.ymlwith 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
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.ymlDetails
This PR was created automatically by the translation action.