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.

Sign up for GitHub

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

Jump to bottom

音声書き出し時に進捗がわかるような表示を実装 #1038

Merged
merged 17 commits into from
Dec 12, 2022
Merged

音声書き出し時に進捗がわかるような表示を実装 #1038

merged 17 commits into from
Dec 12, 2022

Conversation

k-chop
Copy link
Contributor

@k-chop k-chop commented Dec 7, 2022

内容

音声生成を実行してから3秒待つと、書き出しの進捗を表示するモーダルが出るようにしました。

関連 Issue

close #739

スクリーンショット・動画など

対象が1件だけで何%進んでいるか分からない場合

progress1

何%か分かっている場合

（0%で止まると本当に進んでるのか不安になるので、0%の場合はぐるぐるさせるようにしました）

progress2

その他

GENERATE_AND_SAVE_AUDIO は GENERATE_AND_SAVE_ALL_AUDIO の中で呼ばれているので、この中に進捗を更新するロジックを入れてしまうとうまくいきませんでした。
なのでこれらの外側で進捗を管理するようにしています。ALLの方は進捗更新ロジックをcallbackで外から差し込みました。
その代償として、保存ダイアログを開いた瞬間からカウントが始まります・・・なので

音声生成を実行してから3秒待つと

が実は嘘です

@k-chop k-chop mentioned this pull request Dec 7, 2022
Closed
Hiroshiba
Hiroshiba reviewed Dec 7, 2022
View reviewed changes
Copy link
Member

@Hiroshiba Hiroshiba left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PRありがとうございます！！わかりやすい実装でした！！
３秒待ってから表示もとても良いと思います！

finally忘れを防止しつつ、個数を登録する専用のaction関数を実装しなくても良いような設計を考えてみたのでちょっと提案してみました！！

src/components/Dialog.ts Outdated Show resolved Hide resolved
src/components/Dialog.ts Outdated Show resolved Hide resolved
src/store/ui.ts Show resolved Hide resolved
Hiroshiba
Hiroshiba reviewed Dec 8, 2022
View reviewed changes
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/ui.ts Outdated Show resolved Hide resolved
src/store/ui.ts Show resolved Hide resolved
src/components/ProgressDialog.vue Outdated Show resolved Hide resolved
k-chop added 3 commits December 9, 2022 08:21
@k-chop
結果的に同じ実装になってしまったのでSTART_INDETERMINATE_PROGRESSを削除
ede4629 
将来OSのプログレスバーが完全対応されて（もしくはwindowsのみ対応で）導入されることになったら困るかもしれないがその時に対応する
@k-chop
progressの取りうる範囲を明示する
2922196
@k-chop
重複した記述を削除
5d84de4
@k-chop
Copy link
Contributor Author

k-chop commented Dec 8, 2022 •
edited
Loading

レビューありがとうございます 🙏
一通り対応しました！

レビュー時からの変更差分はこちら → fe954f9...5cd951d

Hiroshiba
Hiroshiba approved these changes Dec 9, 2022
View reviewed changes
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！！

@MT224244 さん、もしよかったらふんわりとレビューいただけると心強いです･･･！

MT224244
MT224244 approved these changes Dec 12, 2022
View reviewed changes
Copy link
Contributor

@MT224244 MT224244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM！

@Hiroshiba
Copy link
Member

マージします！PRありがとうございました！！

@Hiroshiba Hiroshiba merged commit 04e0887 into VOICEVOX:main Dec 12, 2022
@k-chop k-chop deleted the issue739/show-progress branch December 13, 2022 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MT224244 MT224244 MT224244 approved these changes

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

音声書き出し時に進捗がわかるような表示をする
3 participants
@k-chop @Hiroshiba @MT224244