Skip to content

fix: 数字と離れた英字を単位読みしない - #32

Merged
owayo merged 1 commit into
mainfrom
fix/issue-29-spaced-alphabet-reading
Oct 12, 2026
Merged

owayo merged 1 commit into
mainfrom
fix/issue-29-spaced-alphabet-reading

Conversation

@owayo

@owayo owayo commented Oct 12, 2026

Copy link
Copy Markdown
Owner

数字と英字の間に空白があるとき、3 F会議室 が「ファラド」と読まれていました。数字の end と英字の start が一致する場合だけ単位読みを付け、空白・タブ・改行で離れた F/A/W を綴り読みへ戻します。

100 km・3 GB・5 V も綴り読みになります。空白のない単位と、3文字以上の既知語にあるカタカナ読みの優先順位は維持します。設計・差分レビューとも妥当と確認し、指摘のあった辞書読みの例外を文書に明記しました。

検証:

  • 修正前に回帰テストが失敗、修正後に通過。既知/未知、全角数字・英字、7種類の隙間、非ゼロ offset、出力への追加を確認。
  • Windows で make ci と同等の format・workspace clippy/test・3 feature 構成の clippy/test が通過。
  • 17,468行を3配布辞書で解析し、3,615,204トークンの全フィールドを比較。対象英字の読み・発音315件以外は一致。
  • 変更前後を交互に3回測定し、各辞書の最小時間の差は -0.5〜-1.5%。
  • astro-sight review: API変更・未対応の呼び出し元なし。差分レビュー: 指摘なし。

Closes #29

空白を読み飛ばすと、離れた数字が英字の直前のトークンになるため、
階や部屋の英字記号を単位として読んでいた。
入力のバイト位置が連続する場合だけ単位読みを付ける。
既知語・未知語、全角英字、空白類と offset の回帰テストを追加する。
@owayo owayo self-assigned this Oct 12, 2026
@owayo
owayo merged commit aa030d8 into main Oct 12, 2026
8 checks passed
@owayo
owayo deleted the fix/issue-29-spaced-alphabet-reading branch October 12, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

数字の後の英字の単位読みが、空白をはさんでも付く (「3 F会議室」の F がファラドになる)

1 participant