-
Notifications
You must be signed in to change notification settings - Fork 310
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
feat: speaker_infoをURLで受け取るようにする #2095
Conversation
eb89791
to
93d7429
Compare
|
93d7429
to
bc3fb8e
Compare
そうかもですが、まあ大丈夫な気がしますね! QImgのloadingをeagerにすれば先に読み込んでくれる気がしましたが、試してみた感じ無理そうでした。 |
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です!!!
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!!
ちょっと文脈難しいのでコメントを追加しつつ、一部コード変更させていただきます!!
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.
ちょこちょこっと変えてみました!!
逆にわかりにくくなってたらすみません。。
(ついでにENGINEのバージョンもあげてみました!)
1つ相談が!
ニーズのある機能が実装されたときにSNSで言及しておりまして、今回のプルリクエストもツイートしたいと思っています。
x.com/search?q=%23VOICEVOX%E9%96%8B%E7%99%BA%E7%8A%B6%E6%B3%81
もしよかったらそこで @sabonerune さんのXアカウントをツイート文に含めて紹介させていただきたいのですが、構わないでしょうか・・・?
こんな感じを予定しています・・・!
#VOICEVOX開発状況
リソース周りの処理を見直し、起動が更に高速になりました🎉(今後のアップデートで実装されます。)
【開発者: @sabonerune】
https://github.com/VOICEVOX/voicevox/pull/2095
@Hiroshiba SNSの掲載、コードの修正ともに問題ありません。 |
マージします! 13時に予約ツイートしました!! エディタの起動高速化のために問題提起から調査・設計・API実装・エディタ側実装ととても大変だったと思います!! @tarepan さんと @sevenc-nanashi さんもありがとうございました!! |
内容
/speaker_info
・/singer_info
の画像・音声をURLで受け取ることによって起動を高速化します。関連 Issue
/speaker_info
と/singer_info
の高速化 voicevox_engine#1129その他
エンジン側の修正が前提です。
speaker_info
のリソースをURLで返すようにする voicevox_engine#1318