Repository navigation
[numpy] Remove the duplicated {index} directive under Matrix Multiplication - #632
Conversation
…cation
The "Matrix Multiplication" section carried the same
`{index} single: NumPy; Matrix Multiplication` directive twice, back to
back, so the general index listed the entry twice. Remove the second.
The French edition already removed its copy (see
QuantEcon/lecture-python-programming.fr#24), which leaves its numpy.md
one directive short of the source; every French sync of numpy.md has
since failed the translation engine's structural-parity check. No prose
or code changes.
Closes #600
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The change is a minimal, well-scoped deletion that leaves exactly one intended {index} directive and does not affect surrounding MyST structure or content.
Review effort: Lite
Findings: None
What changed in this PR
This PR removes a duplicated {index} directive in the NumPy lecture (lectures/numpy.md) under the “Matrix Multiplication” section so the general index no longer lists the entry twice and translation structural-parity checks can align across editions.
Changes:
- Removed the second, back-to-back duplicate
{index} single: NumPy; Matrix Multiplicationdirective, leaving a single directive in place.
| File | Description |
|---|---|
lectures/numpy.md |
Deletes the duplicated {index} directive under “Matrix Multiplication” to prevent duplicate index entries and reduce translation parity mismatches. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
✅ Translation sync completed (zh-cn)Target repo: QuantEcon/lecture-python-programming.zh-cn
|
✅ Translation sync completed (fa)Target repo: QuantEcon/lecture-python-programming.fa
|
✅ Translation sync completed (fr)Target repo: QuantEcon/lecture-python-programming.fr
|
…cation (#60) Mirrors the English source change in QuantEcon/lecture-python-programming#632, by hand: this edition is not on the translation sync. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Removes the second of two identical
{index} single: NumPy; Matrix Multiplicationdirectives inlectures/numpy.md(lines 437–438 atb0b0b56), so the general index stops listing the entry twice. No prose or code changes.Why it matters for the translations. The French edition removed its copy during native-editor review (see QuantEcon/lecture-python-programming.fr#24), so its
numpy.mdhas one directive fewer than the source. The translation engine's structural-parity check compares directive sequences, so every French sync ofnumpy.mdsince has failed (#606, #609, #613), and one edit from #604 (discreteRV→DiscreteRV) never reached the French edition as a result. Simulated against the current engine: French parity passes 6/6 after this change, and the Chinese and Persian editions drop their copies on their next sync 4/4.Closes #600
🤖 Generated with Claude Code