Repository navigation
🌐 [translation-sync] [numpy] Remove the duplicated {index} directive under Matrix Multiplication - #100
🌐 [translation-sync] [numpy] Remove the duplicated {index} directive under Matrix Multiplication#100mmcky wants to merge 2 commits into
Conversation
✅ Translation Quality ReviewVerdict: PASS | Model: claude-sonnet-5 | Date: 2026-11-02 📝 Translation Quality
Summary: The modified 'Matrix Multiplication' section is a faithful, fluent, and correctly formatted translation with no syntax, accuracy, or terminology issues. The Matrix Multiplication section is translated accurately and concisely, preserving code and math exactly Terminology ('multiplication matricielle', 'produit matriciel', 'post-multiplication') matches glossary and standard French usage Section heading and anchor formatting correctly preserved 🔍 Diff Quality
Summary: The target document's changes do not match the source's single, localized removal of a duplicate index directive; instead it removed blank lines across many unrelated sections. Issues:
This review was generated automatically by action-translation review mode. |
There was a problem hiding this comment.
🟢 Approval recommended
No objective issues were identified, as the changes are limited to safe whitespace normalization that preserves structure, a valid wording refinement, and an auto-generated metadata update.
No open issues 🎉
What changed in this PR
This is an automated translation-sync PR that propagates upstream source PR QuantEcon/lecture-python-programming#632 into the French (fr) translation. In the English source the change removed a duplicated {index} directive under the Matrix Multiplication section; because the French file never carried that duplicate, the net effect here is whitespace normalization (removing redundant blank lines) plus a minor wording refinement in the Matrix Multiplication paragraph, along with the usual translation-state metadata bump.
Changes:
- Removes redundant consecutive blank lines before several section headers in
lectures/numpy.md(each header still retains one preceding blank line, so MyST/Markdown structure is preserved). - Rewords one sentence: "tableaux plats … ce que vous voulez" → "tableaux unidimensionnels … ce que l'on veut".
- Updates
.translate/state/numpy.md.ymlmetadata (source SHA, sync date, model, modeUPDATE, tool version).
| File | Description |
|---|---|
lectures/numpy.md |
Removes duplicate blank lines around headers and refines the Matrix Multiplication sentence wording; no structural/markdown breakage. |
.translate/state/numpy.md.yml |
Auto-generated translation-state update (source SHA, synced-at date, model, mode, tool version). |
I reviewed the diff and the surrounding file context. The blank-line removals all preserve a single blank line before each header, so rendering is unaffected, and the wording change is a valid French rendering of the source. No objective issues were found.
Review effort: Balanced
📖 Netlify Preview Ready!Preview URL: https://pr-100--verdant-toffee-3261a8.netlify.app Commit: 📚 Changed LecturesBuild Info
|
|
@Honaminto — could you review this sync before it merges? It follows the English change in QuantEcon/lecture-python-programming#632, which removed a duplicated index entry in
If the new wording reads well to you, an approval is all that's needed; otherwise let us know which version to keep. |
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.