Skip to content

Fix incorrect segmentation of 「星露谷物语」 (Stardew Valley) in s2twp mode#1096

Merged
frankslin merged 3 commits intomasterfrom
copilot/fix-s2twp-translation-error
Apr 12, 2026
Merged

Fix incorrect segmentation of 「星露谷物语」 (Stardew Valley) in s2twp mode#1096
frankslin merged 3 commits intomasterfrom
copilot/fix-s2twp-translation-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 12, 2026

  • Add 星露谷物语 星露谷物語 to data/dictionary/STPhrases.txt
  • Add stardew_valley test case to test/testcases/testcases.json
  • Restore the console_stays_unchanged test case that was accidentally removed

Add 星露谷物语→星露谷物語 to STPhrases.txt so the mmseg segmenter
matches the full game title as one unit, preventing the incorrect
split at 谷物 which was producing 星露穀物語.

Add test case covering s2t, s2tw, s2twp, and s2hk conversions.

Agent-Logs-Url: https://github.com/BYVoid/OpenCC/sessions/45cee4c8-2eca-49e9-bf5c-06bde0c3ae2a

Co-authored-by: frankslin <748873+frankslin@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix s2twp mode incorrect translation of 星露谷物语 Fix incorrect segmentation of 「星露谷物语」 (Stardew Valley) in s2twp mode Apr 12, 2026
Copilot AI requested a review from frankslin April 12, 2026 06:02
@frankslin frankslin marked this pull request as ready for review April 12, 2026 06:52
@frankslin frankslin merged commit cabd32f into master Apr 12, 2026
33 checks passed
@frankslin frankslin deleted the copilot/fix-s2twp-translation-error branch April 12, 2026 07:05
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.

「星露谷物语」 should be 「星露谷物語」, not 「星露穀物語」

2 participants