-
Notifications
You must be signed in to change notification settings - Fork 196
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
追加: 音素 Literal 型 #942
追加: 音素 Literal 型 #942
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
↑のPRと同様に、pydanticで型の保証までできると良いかもと思いました!
まあこちらは直接指定されることがないので、validation無しのLiteralで良いのかも。
_PHONEME_LIST: Sequence[Vowel | Consonant] = ( | ||
_P_LIST1 + _P_LIST2 + _P_LIST3 + _P_LIST4 + _P_LIST5 | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tupleのが意図にあってるかも?
ちょっとこちらで変えさせていただきます! 違ってたらすみません!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うーん! すみません、mypyのことをよく把握していなくてエラーが出てしまいました 🙇
これくらいは型推論してほしい気もしますが、とりあえず型指定しました・・・。
pydanticのvalidate使えばもう少しきれいに書ける気がします。。
内容
音素に Literal 型を追加
VOICEVOX ENGINE ではドメインごとに「音素」があり、それらが相互変換される。
この変換は型レベルで保証が可能である。
そのためにはまず音素を型付けする必要がある。
このような背景から、Literal 型による
Phoneme
音素の型付けを提案します。関連 Issue
#894 の前提条件