Skip to content

Fix Chinese lyrics detection and remove tone suffix from pinyin#2236

Merged
theovilardo merged 1 commit into
PixelPlayerHQ:masterfrom
adlifarizi:fix/chinese-romanization
Jun 4, 2026
Merged

Fix Chinese lyrics detection and remove tone suffix from pinyin#2236
theovilardo merged 1 commit into
PixelPlayerHQ:masterfrom
adlifarizi:fix/chinese-romanization

Conversation

@adlifarizi

Copy link
Copy Markdown
Contributor
  • Fixed a bug where Chinese lyrics were incorrectly identified and romanized as Japanese by checking if Japanese Kana (Hiragana/Katakana) is present in the song before treating CJK Unified Ideographs (Kanji/Hanzi) as Japanese.
  • Removed redundant tonal sandhi logic for and so they correctly fall back to default tone-less pinyin output (yi and bu instead of bu4).

@theovilardo theovilardo merged commit 4d75007 into PixelPlayerHQ:master Jun 4, 2026
5 checks passed
@adlifarizi adlifarizi deleted the fix/chinese-romanization branch June 13, 2026 15:41
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.

2 participants