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
確かに手動なので「補正」より「調整」の方が良いかもです…！
|
|
コメントありがとうございます！！！
こちらの判断すごく助かります 🙇 🙇 🙇 |
|
@sigprogramming 変更しました！！ 今日明日ぐらいでプレビュー版のリリース作ってみたいと思うので、間に合わなそうだったら申し訳ないのですがレビューなしでマージさせていただこうと思います 🙇 |
Hiroshiba
commented
Mar 6, 2024
sevenc-nanashi
pushed a commit
to sevenc-nanashi/voicevox
that referenced
this pull request
Mar 6, 2024
* 音高補正にしつつ開発時のみ機能に * VoiceKey→GuidePitch * とりあえず実装としては完成 * keyRangeAdjustmentに * (note|guide)KeyShiftを消し、補正→調整にする * keyShiftを全てKeyRangeAdjustmentへ
Hiroshiba
added a commit
that referenced
this pull request
Mar 18, 2024
* Improve: レンダリング順を改善 * Fix: 値を修正 * Fix: toEqualを使う * 音域補正用のパラメーターを増やしつつ、開発時のみの機能に (#1902) * 音高補正にしつつ開発時のみ機能に * VoiceKey→GuidePitch * とりあえず実装としては完成 * keyRangeAdjustmentに * (note|guide)KeyShiftを消し、補正→調整にする * keyShiftを全てKeyRangeAdjustmentへ * Update: mainに追従 * Improve: レビューを反映 Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com> * Change: findPriorPhrases -> selectPriorPhrases * Delete: 不要な型アサーションを削除 Co-Authored-By: sigprogramming <sigprogramming@users.noreply.github.com> * Add: 型アサーションを追加 * Update src/sing/domain.ts * Update src/sing/domain.ts --------- Co-authored-by: Hiroshiba <hihokaruta@gmail.com> Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com> Co-authored-by: sigprogramming <sigprogramming@users.noreply.github.com>
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.
内容
色々考えた結果、音域補正用のパラメーターを増やすのが良さそうだったので増やしました。
ついでにリリースに向けて開発時のみの機能にしました。
変更点は以下の４点です。
voiceKeyShiftをguideKeyShiftにkeyRangeAdjustmentパラメータを追加notesKeyShiftとguideKeyShiftの２つともに干渉する値なので別定義が必要だった関連 Issue
resolve #1899
その他
迷ってることがいくつかあります。
keyRangeAdjustmentは直訳すると音域"調整"だけどこれで良いかCorrectionと迷ったのですが、何を持って「正しい」とするのかわからなかったので調整にしてみましたnotesKeyShiftとguideKeyShiftを削除するかそのままにするかguideKeyShiftは将来的に使わない可能性がありそうguideKeyShiftが議論の結果keyRangeAdjustmentに変わったという感じかも