We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
split_mora()
要望: split_mora() 関数の二分割によるリファクタリング
モーラを音素に分割する split_mora() 関数は現在、以下の3箇所で利用されている。
voicevox_engine/voicevox_engine/synthesis_engine/synthesis_engine.py
Line 191 in 38be022
Line 310 in 38be022
Lines 435 to 439 in 38be022
このように、2/3のユースケースでは vowel_indexes_data 返り値のみを利用している。
vowel_indexes_data
split_mora() 内部では早期に vowel_indexes_data を計算しており、これを取り出すだけなら以後の全演算が不要である。
Lines 83 to 86 in 38be022
また split_mora() という関数名から母音インデックス取り出しを想起することは難しい。
上記の理由から母音インデックス取り出し関数の切り出しによるリファクタリングを提案します。
_find_vowel_indexes()
0.14.10
The text was updated successfully, but these errors were encountered:
提案ありがとうございます! 方針として良いのかなと思いました!!
Sorry, something went wrong.
👍 #790 に依存しているため、当該PRマージ後に着手します。
replace_phoneme_length()
Successfully merging a pull request may close this issue.
内容
要望:
split_mora()
関数の二分割によるリファクタリングモーラを音素に分割する
split_mora()
関数は現在、以下の3箇所で利用されている。voicevox_engine/voicevox_engine/synthesis_engine/synthesis_engine.py
Line 191 in 38be022
voicevox_engine/voicevox_engine/synthesis_engine/synthesis_engine.py
Line 310 in 38be022
voicevox_engine/voicevox_engine/synthesis_engine/synthesis_engine.py
Lines 435 to 439 in 38be022
このように、2/3のユースケースでは
vowel_indexes_data
返り値のみを利用している。split_mora()
内部では早期にvowel_indexes_data
を計算しており、これを取り出すだけなら以後の全演算が不要である。voicevox_engine/voicevox_engine/synthesis_engine/synthesis_engine.py
Lines 83 to 86 in 38be022
また
split_mora()
という関数名から母音インデックス取り出しを想起することは難しい。上記の理由から母音インデックス取り出し関数の切り出しによるリファクタリングを提案します。
Pros 良くなる点
Cons 悪くなる点
実現方法
_find_vowel_indexes()
関数への機能切り出しVOICEVOXのバージョン
0.14.10
OSの種類/ディストリ/バージョン
The text was updated successfully, but these errors were encountered: