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

[ソング] PhraseKeyに型をつける #1856

Closed
Hiroshiba opened this issue Feb 19, 2024 · 3 comments
Closed

[ソング] PhraseKeyに型をつける #1856

Hiroshiba opened this issue Feb 19, 2024 · 3 comments
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

ソングはPhraseという区切りがあり、対応するPhraseKeyをキャッシュなどに利用しています。

Pros 良くなる点

型間違いを起こしづらくなり、コーディングが楽になる

実現方法

たぶんこんな感じで定義して、
https://github.com/VOICEVOX/voicevox/pull/1217/files#diff-a2e928a7b8428dba5f753cc8014e39204ccb3aaf1049ba8567f4b59af69defc7R26-R28
あとはいろんなところのstringをPhraseKeyに置き換える感じだと思います。
SingingStoreStateのphrases: Map<string, Phrase>のstringとか。

phraseKeyでコード全体検索をすると見つけやすいはず。
トーク側のAccentPhraseと間違いやすいかもなので注意が必要かも

その他

がマージされてからのがやりやすそう。

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク labels Feb 19, 2024
@Hiroshiba Hiroshiba changed the title [ソング] PhraseKyに型をつける [ソング] PhraseKeyに型をつける Feb 19, 2024
@jdkfx
Copy link
Contributor

jdkfx commented Nov 3, 2024

この内容ってこちらで解決されていませんか?
間違っていたらすみません!

https://github.com/VOICEVOX/voicevox/pull/2248/files#diff-d26d5d8607099d0eb54ebe12db20ed3f7d1c9d9593727df50d16feef55a6bb4a

@sevenc-nanashi
Copy link
Member

確かにできてるかも、ありがとうございます

@Hiroshiba
Copy link
Member Author

報告ありがとうございます!!

結構closeし忘れてるのあると思うので、また見つけたらぜひコメントいただけると・・・!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

No branches or pull requests

3 participants