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

feat: speaker_infoをURLで受け取るようにする #2095

Merged
merged 6 commits into from
Jun 28, 2024

Conversation

sabonerune
Copy link
Contributor

@sabonerune sabonerune commented May 24, 2024

内容

/speaker_info/singer_infoの画像・音声をURLで受け取ることによって起動を高速化します。

関連 Issue

その他

エンジン側の修正が前提です。

@sabonerune
Copy link
Contributor Author

CharacterButtonを押したとき画像キャッシュがない場合、わずかに画像の取得に時間がかかる感じがします。
ただ、アニメーションを有効にしても意味がなさそうで自分の技術ではどうにもならなそう、一瞬なので許容範囲と考えとりあえずオープンします。

@sabonerune sabonerune marked this pull request as ready for review June 25, 2024 11:17
@sabonerune sabonerune requested a review from a team as a code owner June 25, 2024 11:17
@sabonerune sabonerune requested review from Hiroshiba and removed request for a team June 25, 2024 11:17
@Hiroshiba
Copy link
Member

CharacterButtonを押したとき画像キャッシュがない場合、わずかに画像の取得に時間がかかる感じがします。

そうかもですが、まあ大丈夫な気がしますね!
気になる感じであれば、先にfetchすることでキャッシュが作れるなら先にfetchしても良いかも。

QImgのloadingをeagerにすれば先に読み込んでくれる気がしましたが、試してみた感じ無理そうでした。
たぶんQMenuが遅延でDOM作るからな気がします。
やはりあらかじめレンダリングしておいて、クリックしたときにクリックした場所に表示されるようにしたい。。。
(たぶんこのissueと同じ文脈)

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.

ほぼLGTMです!!!

src/store/audio.ts Outdated Show resolved Hide resolved
src/store/audio.ts Outdated Show resolved Hide resolved
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.

LGTM!!

ちょっと文脈難しいのでコメントを追加しつつ、一部コード変更させていただきます!!

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.

ちょこちょこっと変えてみました!!
逆にわかりにくくなってたらすみません。。

(ついでに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

@sabonerune
Copy link
Contributor Author

@Hiroshiba SNSの掲載、コードの修正ともに問題ありません。
ありがとうございます。

@Hiroshiba
Copy link
Member

マージします! 13時に予約ツイートしました!!

エディタの起動高速化のために問題提起から調査・設計・API実装・エディタ側実装ととても大変だったと思います!!
おかげで起動はかなり早くなったと思います!!
改めてありがとうございました!!!

@tarepan さんと @sevenc-nanashi さんもありがとうございました!!

@Hiroshiba Hiroshiba merged commit 1deb8b4 into VOICEVOX:main Jun 28, 2024
9 checks passed
@sabonerune sabonerune deleted the feat/url-infos branch June 29, 2024 01:54
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.

2 participants