Repository navigation
Apply the #57 answer to numpy: commas on case-marked -ഉം pairs (2 places) - #59
Conversation
…airs Adisankar ruled on #57 (answer 1.2) that two items carrying a case ending as well as -ഉം always take a comma, as plain -ഉം pairs do. Two pairs on the reviewed numpy page were still bare: - line 840: `a`-യിലും, `b`-യിലും - line 902: speed-ന്റെയും, memory-യുടെയും Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
📖 HTML build - view preview (3a875d9) Translated pages in this PR: |
There was a problem hiding this comment.
🟢 Approval recommended
The two edits correctly and completely apply an established, codebase-verified comma convention with no missed pairs and no functional impact.
No open issues 🎉
What changed in this PR
This PR applies a confirmed Malayalam translation style decision (from issue #57, answer 1.2) to the already-published numpy lecture: it inserts a comma between two coordinated items that each carry a case ending plus -ഉം. It is a minimal typographic/style fix to the Malayalam translation with no code or functional changes, bringing the numpy page into line with the convention already used in matplotlib and python_by_example.
Changes:
- Line 840:
`a`-യിലും `b`-യിലും→`a`-യിലും, `b`-യിലും(added comma). - Line 902:
speed-ന്റെയും memory-യുടെയും→speed-ന്റെയും, memory-യുടെയും(added comma).
| File | Description |
|---|---|
| lectures/numpy.md | Adds a comma between two case-marked -ഉം items in two places (lines 840 and 902) to match the established "always a comma" convention. |
Review effort: Balanced
|
approved. both reads correctly |
@adisankarmt — thank you for the answers on #57. Your answer 1.2 (always a comma) applies to two places on the
numpypage that is already on the site, so this small PR brings them into line. Nothing else changes. It needs only a quick look from you: approve it if both read correctly, or tell us which to put back.From your answer 1.2: always a comma between case-marked
-ഉംitems`a`-യിലും `b`-യിലും`a`-യിലും, `b`-യിലുംspeed-ന്റെയും memory-യുടെയുംspeed-ന്റെയും, memory-യുടെയുംThese are the two pairs #57 mentioned, which you left without a comma in round 4. A search of the other reviewed pages found no more pairs of this kind.
For @mmcky: the search also found shapes the engine's comma repair must leave alone, and these are now its test cases: a pair with a further ending on the second item, such as
rows-ലും columns-ലുമായി(pandas, line 161, the editor's own round-5 text); verb pairs such asrepresent ചെയ്യാനും analyze ചെയ്യാനും; and single items that mean "all" or "any", such asഎല്ലാ plots-നെയും. The tool-side change is QuantEcon/action-translation#403.🤖 Generated with Claude Code