[ソング] ピッチ編集に合わせてボリュームを自動的に更新する #2015
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
y-chan
requested review from
Hiroshiba,
romot-co and
sigprogramming
and removed request for
a team
|
(sigさんromotさんをレビュワーにアサインさせていただきました...!見ていただけると大変助かります 🙇 ) |
Hiroshiba
reviewed
Apr 26, 2024
|
volumeを生成するときの |
Hiroshiba
reviewed
Apr 26, 2024
sigprogramming
approved these changes
Apr 26, 2024
Hiroshiba
approved these changes
Apr 26, 2024
Comment on lines
+935
to
+942
| const notesForRequestToEngine = createNotesForRequestToEngine( | ||
| notes, | ||
| tempos, | ||
| tpqn, | ||
| keyRangeAdjustment, | ||
| frameRate, | ||
| restDurationSeconds, | ||
| ); |
There was a problem hiding this comment.
これ2回実行してますが、1回実行してその結果を関数に渡す形でも良い気がしました！
もちろんエンジンに投げるスコアが必ず同じならですが。（今のところ必ず同じはず。）
Comment on lines
+1345
to
+1347
| // ピッチ編集を適用したクエリから音量を作る | ||
| // 音量値はAPIを叩く毎に変わるので、calc hashしたあとに音量を取得している | ||
| const notesForRequestToEngine = createNotesForRequestToEngine( |
There was a problem hiding this comment.
これ多分本来は合成直前ではなく、ピッチ編集するたびに実行にした方が良さそうなんですよね。
入力が同じボリュームの計算を2回行っちゃってるので。
ただまぁボリュームの計算は重くないので、一旦毎回2回計算しちゃう形で良さそう。
でもなんか忘れそうなのでissue化しておくと安心なのかなとか思いました！
|
こちら、声量調整パラメータの最適用を忘れていることに気づきました！！ |
sevenc-nanashi
pushed a commit
to sevenc-nanashi/voicevox
that referenced
this pull request
Apr 28, 2024
* update openapi * add volume update process * fix miss * update comment * remove comment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
Note
VOICEVOX/voicevox_engine#1187 のマージが必要です
ピッチ編集後のレンダー時に、音量を更新します。
なお、レンダー時の音声合成の直前で音量を更新するため、その音量は現状プロジェクトに保存されません。
音量を操作できるようにする前に、保存されるようにする必要があります。
関連 Issue
ref VOICEVOX/voicevox_engine#1187
スクリーンショット・動画など
その他
今最速でできる一番バグが起きにくい形での実装を考えました
本当は色々考えてプロジェクトファイルへ音量を書き出すべきなのですが、UNDO/REDO回りがややこしくなりそうだったので、諦めました...