Skip to content

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です!!!

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
@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