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

追加: 文内無音倍率 #2352

Merged
merged 14 commits into from
Nov 15, 2024
Merged

追加: 文内無音倍率 #2352

merged 14 commits into from
Nov 15, 2024

Conversation

X-20A
Copy link
Contributor

@X-20A X-20A commented Nov 12, 2024

内容

ほったらかしてしまって申し訳ないです
倍率の方だけ実装してみましたが
・後方互換に関する仕様について把握していないので不安が残る
・「0.19.1からのマイグレーション時にハミング・ソングスタイル由来のデフォルトプリセットを削除できている」テストに引っかかる
という状況です

関連 Issue

#2083

@X-20A X-20A requested a review from a team as a code owner November 12, 2024 11:13
@X-20A X-20A requested review from Hiroshiba and removed request for a team November 12, 2024 11:13
@sevenc-nanashi
Copy link
Member

設定のマイグレーションがないのでエラーが起きてるのかと。src/backend/common下にマイグレーション周りのコードがあります

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.

うおーシンプル!!!すごく良い感じですね!!

マイグレーション部分で落ちてるっぽいのは、結構難しいと思うのでこっちでやっておくことも可能です!
今たぶんif (semver.satisfies(projectAppVersion, "<0.4", semverSatisfiesOptions)) {のとこにマイグレーション用のコードを書かれてるので、
新しく<0.22用のifを追加して、そこにそんな感じの処理を書けばOKだと思います!

@X-20A
Copy link
Contributor Author

X-20A commented Nov 12, 2024

正直読んでもよく分からないのでお願いしたいです...

@Hiroshiba
Copy link
Member

@X-20A
こちらで変更してみました!! よかったらプルリク取り込んでいただければ!

もちろん更に変更いただいたりしても全く問題ないです 🙏

@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Nov 13, 2024

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:7f72bc0

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

ちょっと1つPR出させていただきました!
それプラスコメントの内容ができれば完成だと思います!!!

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

実際に試してみました! 良さそう!!!
あともう少し・・・!!

generateLabFromAudioQueryも変更する必要があることに気づきました!
たぶんこちらだと思います!
https://github.com/X-20A/voicevox/blob/7f72bc095292b24e01fea4f83919450c656e2f5a/src/store/audioGenerate.ts#L103-L104

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

エンジンの仕様変更からエディタの実装まで、ありがとうございました!!!
かなりニーズのある機能だと思うのでリストが楽しみです!!

絶対値の無音時間指定ですが、実装すべきかどうかをしばらく様子見させていただいてもよろしいでしょうか 🙇

理由はUXの設計がかなり難しいと感じているためです。
倍率と設定変更可能にする・絶対時間パラメータを「無指定にする」操作を作る・指定地は値を破壊的に置き換えるなど、結構大変だなと。
一旦倍率指定で需要はかなり満たされると思うので、それでもなお需要が生まれるかどうかを様子見させていただければと思っています。
もしちょっと必要そうだと思ったらissueを作るか既存のissueを再利用して呼びかけたいと思いますので、もしよかったら 🙏

あと他にもVOICEVOXに待ち望まれている機能は結構あったりします!
ソングで.labファイル(口パクファイル)を出力するタスクや、
トーク・ソングどちらのエディターを開いていたか保存し、起動時にそちらを開くタスクなど、
ご興味あればぜひ・・・・・!!!


もう1つ相談があります!

ニーズのある機能が実装されたときにSNSで言及しておりまして、今回の機能が実装されたこともポストしたいと思っています。
x.com/search?q=%23VOICEVOX%E9%96%8B%E7%99%BA%E7%8A%B6%E6%B3%81
もしよかったらそこで @X-20A さんのXアカウントをツイート文に含めて紹介させていただきたいのですが、いかがでしょうか・・・?
アカウントがなければ、@は無しでそのまま X-20A と記載したいと思います 🙏

ちなみにこんな感じを予定しています・・・!

#VOICEVOX開発状況 
一括で文内の無音(ポーズ)時間を制御できるようになりました 🎉(今後のアップデートで実装されます。)
【開発者: @~~~】
https://github.com/VOICEVOX/voicevox/pull/2352

Comment on lines +744 to +745
newAudioItem.query.pauseLengthScale =
baseAudioItem.query.pauseLengthScale;
Copy link
Member

@Hiroshiba Hiroshiba Nov 15, 2024

Choose a reason for hiding this comment

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

あ、ここ順番変わってなそうですね!
ちょっとこちらで変えさせていただこうと思います!

(追記:変更忘れていました・・・。)

@Hiroshiba Hiroshiba changed the title 追加: 文内無音倍率(新規ブランチ) 追加: 文内無音倍率 Nov 15, 2024
@Hiroshiba Hiroshiba merged commit 258c791 into VOICEVOX:main Nov 15, 2024
@X-20A
Copy link
Contributor Author

X-20A commented Nov 15, 2024

ありがとうございます、うれしいです
アカウントは@momemiでお願いします
他のissueについてはコードを見ながら考えようと思います

@X-20A X-20A deleted the addPLS branch November 15, 2024 16:50
@Hiroshiba
Copy link
Member

こちらこそありがとうございます!
ツイートさせていただきました! https://x.com/voicevox_pj/status/1857624908857286765

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.

3 participants