ja: fix the rule vocabulary outside navigate.yaml - #748
Merged
Conversation
Five rule files still carried seed mistranslations of the same kind daisy#746 cleared from navigate.yaml: doubled words, furniture and equipment where mathematics was meant, and English word order glued between Japanese tokens. ClearSpeak multi-line labels: case was ケースケース, step was ステップステップ, equation was 設備 (equipment) and line was ライン; now ケース, ステップ, 式 and 行, in both the count rule and the per-line rule. SharedRules/default.yaml: "table with 3 rows and 4 columns" was テーブルと 3 行 および 4 コラム (the noun first, as in English, and コラム is a newspaper column); it is now 3 行 および 4 列 の表. The menclose line notation was ライン + 左から/アクセス/トップトップ/ボトム ("line" + "from the left" / "access" / "top top" / "bottom"); it is now 左に/右に/上に/下に + 線, side first and the noun last. "with 2 prescripts" was 付き 2 プレスクリプト; it is now 2 個の前置き添字 付き, and the overflow branch's 終了原稿 ("end manuscript", the same seed error family as 事前原稿 in navigate) is 前置き添字終了, with "and alternating prescripts" as および交互の前置き添字. These three are line reorders inside a rule, no lines added or removed. overview.yaml: "the 2 by 3 table" ended in テーブル (furniture); it now ends in の表, the way 行列 is already read. definitions.yaml: "change in" (Δ) was 変更点 ("modification point"), now 変化量. SharedRules/calculus.yaml: ラ・プラハシアン was the en TTS respelling "LahPlahsian" copied as katakana; definitions.yaml already says ラプラシアン, so it does now too. Per AGENTS.md the 22 verified lines are promoted t: -> T:. audit-translations: untranslated text 3674 -> 3652, rule differences 28 -> 28, missing/extra rules 0 -> 0. Two tests cover the multi-line case label and the menclose side line. Left alone on purpose: "such that" (そのようなこと, 4 sites), round (ラウンド値) and fenced-group (フェンスグループ) need a wording decision rather than a swap. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016JCoREgn1pJzcbdnUx4iuh
…matrix column)
Follow-up inside the same commit's scope, found by re-reading the rules the
first pass touched.
The postscripts branch is the mirror of the prescripts one and had the same
two faults: 付き N 投稿原稿 read "with N postscripts" in English order and
投稿原稿 is a manuscript submitted to a magazine; the overflow branch said
スクリプトの変更と変更 ("script change and change") and エンドスクリプト. They
now read N 個の後置き添字 付き, および交互の添字 and 添字終了, matching the
prescripts wording.
menclose's own enclosure markers were transliterations: エンクロージャー and
エンドエンクロージャ. The marker is emitted before the content, so a verb
phrase does not work; it is 囲み ... 囲み終了, the marker/end-marker shape this
language already uses (上付き ... 上付き終了). The side pause moved with the
noun, so the announcement is 左に 線, ... rather than 左に, 線 ....
The matrix column rule still said コラム after the table rule was fixed; it is
now 列 like the rest.
audit-translations: untranslated text 3674 -> 3645, rule differences 28 -> 28,
missing/extra rules 0 -> 0.
Left alone on purpose: ロングディビジョンシンボル ("long division symbol")
needs a wording decision, as do such-that, round and fenced-group.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016JCoREgn1pJzcbdnUx4iuh
Caught in review. unicode-full.yaml was already inconsistent with itself: the verbose branch said ラプラシアン の while the two terse branches still said ラ・プラハシアン, the en TTS respelling "LahPlahsian" copied as katakana. All three now say ラプラシアン, matching definitions.yaml, unicode.yaml and SharedRules/calculus.yaml. This file is otherwise being left for the MathPlayer-derived seed; these three lines are only the reading of one word already decided elsewhere, so a reseed can overwrite them harmlessly. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016JCoREgn1pJzcbdnUx4iuh
daisy#747 has landed, so SharedRules/general.yaml is free to touch. It carries the SimpleSpeak copy of the multi-line labels this branch already fixed in ClearSpeak_Rules.yaml, and leaving it would have made the two styles disagree: ClearSpeak saying ケース and SimpleSpeak ケースケース for the same piecewise. Both rules in general.yaml now read ケース, 式 and 行, and the test asserts the SimpleSpeak path as well as ClearSpeak. The rest of general.yaml (vector, end, sub, the chemistry terms) is a separate batch, not this one. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016JCoREgn1pJzcbdnUx4iuh
yasumorishima
force-pushed
the
ja-rules-vocabulary
branch
from
September 3, 2026 23:32
751458c to
4d22a87
Compare
The SimpleSpeak path classifies this table as a system of equations rather than cases, so it reads 2 式 / 式 1 / 式 2 through SharedRules/general.yaml while ClearSpeak with MultiLineLabel=Case reads ケース. Both are now asserted. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016JCoREgn1pJzcbdnUx4iuh
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
2 participants
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.
The same class of seed errors #746 cleared out of
navigate.yaml, in the five rule files it did not touch: doubled words, furniture and equipment where mathematics was meant, English word order glued between Japanese tokens, and en TTS respellings copied as katakana.ClearSpeak multi-line labels (both the count rule and the per-line rule): case was ケースケース, step was ステップステップ, equation was 設備 (equipment), line was ライン. Now ケース, ステップ, 式, 行.
SharedRules/default.yaml
overview.yaml: "the 2 by 3 table" ended in テーブル; now の表, the way 行列 is already read.
definitions.yaml: "change in" (Δ) was 変更点 ("modification point"); now 変化量.
calculus.yaml / unicode-full.yaml: ラ・プラハシアン is en's TTS respelling "LahPlahsian" as katakana.
definitions.yamlandunicode.yamlalready said ラプラシアン, andunicode-full.yamldisagreed with itself line to line; all paths now read ラプラシアン.The three reorders are line moves inside one rule — no lines added or removed. Per AGENTS.md only verified lines are promoted
t:→T:.audit-translations ja: untranslated text 3674 → 3645, rule differences 28 → 28, missing/extra rules 0 → 0. Two tests cover the multi-line case label and the menclose side line.Left alone on purpose, because each needs a wording decision rather than a swap:
such that(そのようなこと),round(ラウンド値),fenced-group(フェンスグループ) andlong division symbol(ロングディビジョンシンボル).SharedRules/general.yamlis untouched here to stay clear of #747.🤖 Generated with Claude Code