Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: mora_mapping.py にOpenJTalkに無いKanaを追加 #1473

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

rokujyushi
Copy link

@rokujyushi rokujyushi commented Oct 12, 2024

内容

mora_mapping.py にOpenJTalkに無い以下のKanaを追加

"キィ","ギィ","クァ","クィ","クゥ","クェ","クォ","グァ","グィ","グゥ","グェ","グォ",
"ヂェ","ヂャ","ヂュ","ヂョ","テェ","ニィ","ヒィ","ビィ","ピィ","ミィ","リィ","ウゥ"

関連 Issue

close #1463

その他

  • 関連PR
    変更:歌詞の入力内容チェックの正規表現を修正 voicevox#2293

  • 以下のテストでエラー

    • test_mora2text_injective
    • test_count_mora_x
  • デバッグ実行で動作確認
    追加したKanaの一部の組み合わせでmora_kana_to_mora_phonemesの中身がおかしくなる
    "クィ","クゥ","クェ","クォ"などで辞書に一致するキーがなくエラーになり合成に失敗する

imageimage

  • 試したこと
    • VSCodeの再起動
      2回目のデバッグ実行で再発
    • _mora_list_minimum変数の記述内容見直し

テスト用プロジェクト.zip

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あーーーーーーーーーーーーー原因がわかりました!!!!!!!!

Comment on lines 375 to 378
{"クィ", "kw", "i"},
{"クゥ", "kw", "u"},
{"クェ", "kw", "e"},
{"クォ", "kw", "o"},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここ、{}ではなく()ですね!!!

ちなみに静的解析のpysen run format lintとかが便利かもです 🙏

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.

モーラマッピングにOpenJTalkに無いKanaを追加する。
2 participants