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>
Contributor
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.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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