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

モーフィング機能の制限に対応(改) #1145

Merged
merged 13 commits into from
Jan 25, 2023

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Jan 24, 2023

内容

mainブランチのUIとの差分として

  • モーフィング可能なスタイル+既に選択されているキャラクターしか表示されない
  • 複数エンジンの時に跨ったエンジンのキャラを表示しない

が生じます。

動画

_VOICEVOX.-.Ver.999.999.999.-.CPU.2023-01-25.06-53-06.mp4

その他

@Segu-g さんのモーフィング機能制限PRを引き継いで作ったPRです。

PR間の違いは

  • モーフィング可能テーブルのキャッシュを無限にした
  • PAIR関数をなくし、morphable targetに含まれるスタイルを残す形にした

@Hiroshiba Hiroshiba requested a review from a team as a code owner January 24, 2023 21:54
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team January 24, 2023 21:54
@Hiroshiba
Copy link
Member Author

@Segu-g さんほんとにありがとうございます!!!

@sabonerune さんすみません、もしよかったらUI触ってみたりして頂いても良いでしょうか 🙇

@sabonerune
Copy link
Contributor

とりあえず軽く触っただけの段階ですが取り急ぎ

AudioCellを削除するときここでエラーが発生しています。

watch(
() => [audioItem.value.engineId, audioItem.value.styleId],
() => {
if (
audioItem.value.engineId != undefined &&
audioItem.value.styleId != undefined

あとはもち子さんがスタイルが1つしかないが自分自身しか選択ができないという点も気になりますがこれは後回しでもよさそうですね…

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 24, 2023

@sabonerune おおほんとですね!!!ありがとうございます!!修正しました!

たしかに1スタイルだけ表示になりますね・・・。
自分自身のスタイルは非表示にしようかとちょっと考えましたが、ベースのスタイルを変えてモーフィングのスタイルがかぶったときにUIどうすれば良いのか自明じゃなさそうなのでパスしました!

@sabonerune
Copy link
Contributor

あとはSHAREVOXエンジンが/morphable_targetsに非対応のためエラーが発生して選択可能なキャラクターがないことになってしまいますがこれは正式リリース前なので考慮しなくて平気でしょうか。

思いつく限りの操作を試してみましたが多分問題ないと思います。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 25, 2023

ですね、正式リリース前の機能なので追従をお待ちする形が良いのかなと思いました!
(共有です @y-chan

動作レビューありがとうございます!!!
マージします!

@Hiroshiba Hiroshiba merged commit d053d98 into VOICEVOX:main Jan 25, 2023
@Hiroshiba Hiroshiba deleted the feature/morphing-permission branch January 25, 2023 15:21
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.

2 participants