既存キャラクターの音域調整量をマジックナンバーとして埋め込み、自動入力する #2028
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
|
よくソング周りのプルリクエストをいただく方に共有です 🙇 音域調整量のワークアラウンドを実装してみました。時間がなかったためマジックナンバーになってます。 |
Hiroshiba
commented
Apr 26, 2024
Comment on lines
221
to
+227
| SET_SINGER: { | ||
| mutation(state, { singer }: { singer?: Singer }) { | ||
| // 歌手をセットする。 | ||
| // withRelatedがtrueの場合、関連する情報もセットする。 | ||
| mutation( | ||
| state, | ||
| { singer, withRelated }: { singer?: Singer; withRelated?: boolean }, | ||
| ) { |
There was a problem hiding this comment.
set singerを少し改造して、セットする時に関連データも一緒にセットするかどうかを選べるようにしました。
デフォルトは普通のセッターです。ちなみに普通にsingerだけsetしてるのはプロジェクトファイル読み込みの部分だけです。
今は音域調整料だけですが、将来的には声量調整量とか、まあその他もろもろとかを突っ込んでいければいいのかなと思っています。
|
バグがないかかなり怖いですが、テストが通ったらマージしたいと思います 🙇 |
sevenc-nanashi
pushed a commit
to sevenc-nanashi/voicevox
that referenced
this pull request
Apr 28, 2024
* 音量域→声量 * 音域自動調整ワークアラウンド 実装しないほうが良いかもなやつ * stash * withRelated追加 * 音域の値追加 * 説明追加 * 説明を追加 * COMMAND_SET_SINGERが冗長だった * だいぶ見づらいので修正
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.
内容
音域調整量は現状手動入力です。
音域の基準値をエンジン側に持たせてそこから音域調整量を計算する予定でしたが、時間が足りず手が回りませんでした。
正式リリースの時にはあって欲しかった機能だったので、無理やりですがマジックナンバーとしてテーブルを持ち、値を引っ張ってくるようにしました。
それに伴って使い方の方でも説明を変えておきました。
関連 Issue
その他
声量調整量は未設定です。
（経験的に3や5が良いのでそうしようかと思いましたが、保留で。。。）
アイデアですが、音域調整という相対的な量ではなく、どの音域に合わせるかという絶対的な値を指定する形にすると分かりやすそうだなと思いました。
歌い方の歌手によっても変わらないし、「G4」などとしておけばどのあたりなのかも分かりやすそうだし。
（初心者の人にとってはどれがG4かわからないかもしれませんが、まあアイコンとか表示してあげれば良さそう）