Skip to content

t2s: 乾斷食 → 干断食, 乾紅 → 干红 - #1429

Merged
frankslin merged 2 commits into
masterfrom
copilot/add-t2s-mode-conversion-rules
Jul 15, 2026
Merged

frankslin merged 2 commits into
masterfrom
copilot/add-t2s-mode-conversion-rules

Conversation

Copilot AI commented Jul 15, 2026 •

Copy link
Copy Markdown
Contributor

In t2s and related configs, 乾斷食 (dry fasting) was incorrectly converted to 乾断食 instead of 干断食, and 乾紅 (dry red wine) only produced 乾红 while missing the more common 干红 form.

Changes

data/dictionary/TSPhrases.txt

  • Add 乾斷食 → 干断食: the longer phrase takes longest-prefix-match priority over the existing 乾斷 → 乾断 entry, producing the correct simplified form for the dry-fasting context
  • Change 乾紅 → 乾红 to 乾紅 → 干红 乾红: promotes 干红 as the default candidate (dry red wine usage) while keeping 乾红 (deep red color, read qián hóng) as an alternative

test/testcases/testcases.json

  • Add BYVoid_OpenCC_Issue_621_qian_duan_shi: asserts 乾斷食 → 干断食 across t2s, tw2s, hk2s, tw2sp, hk2sp
  • Add BYVoid_OpenCC_Issue_621_qian_hong: asserts 乾紅 → 干红 across the same five configs

Copilot AI linked an issue Jul 15, 2026 that may be closed by this pull request
Add TSPhrases entry for 乾斷食 → 干断食 so the dry-fasting term
takes priority over the shorter 乾斷 → 乾断 match. Update 乾紅
to carry 干红 as the primary candidate (dry red wine) alongside
乾红 as an alternative. Add corresponding test cases for t2s,
tw2s, hk2s, tw2sp, and hk2sp.
Copilot AI changed the title [WIP] Add t2s mode conversion rules for 乾斷 and 乾紅 Fix t2s over-conversion: 乾斷食 → 干断食, 乾紅 → 干红 Jul 15, 2026
Copilot AI requested a review from frankslin July 15, 2026 00:42
@frankslin
frankslin marked this pull request as ready for review July 15, 2026 00:50
@frankslin frankslin changed the title Fix t2s over-conversion: 乾斷食 → 干断食, 乾紅 → 干红 t2s: 乾斷食 → 干断食, 乾紅 → 干红 Jul 15, 2026
@frankslin
frankslin merged commit 0137a24 into master Jul 15, 2026
43 checks passed
@frankslin
frankslin deleted the copilot/add-t2s-mode-conversion-rules branch July 15, 2026 02:37
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.

TS 可能的过度转换:乾斷,乾紅

2 participants