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

話者ごとのスタイルの中のボイスサンプルやアイコンを必須じゃなくしたい #1051

Open
Hiroshiba opened this issue Feb 6, 2024 · 4 comments · May be fixed by #1052
Labels
機能向上 状態:実装 実装をおこなっている状態

Comments

@Hiroshiba
Copy link
Member

内容

話者ごとのスタイルの中のボイスサンプルやアイコンを必須じゃなくしたいです。
portrait(立ち絵)はスタイルごとに必須じゃないので、それに合わせたい形です。

Pros 良くなる点

mockなどを簡単に追加できる

Cons 悪くなる点

エディタ側も対応しないといけない

実現方法

  • スタイルごとのvoice_samplesとiconをOptionalにする
  • 話者ごとのiconを必須にする

その他

本当にとても限定的なのですが、これで試作した音声ライブラリをエディタから試すのがとても簡単になります・・・。
ちょっとエディタまで巻き込んでやらないとなので少し大変ではあるのですが、頑張りたいです。。

@sabonerune
Copy link
Contributor

開発を楽にすることが主目的ならファイルがない場合はフォールバック用のファイルを返すのはどうでしょうか?
また、ボイスサンプルの方はAPIの定義上、空であることが許容されています。(エディタの方でサンプルが3つ以上あることを想定している方が問題な気がします)

@Hiroshiba
Copy link
Member Author

おお、なるほどです!!! かなり良いのではと感じました!!
ボイスサンプルはたしかに存在しないときに空にしてしまっても良いかもですね。(今は3つないとエンジン側がエラーになる)

@tarepan
Copy link
Contributor

tarepan commented Feb 20, 2024

ref #337

@Hiroshiba
Copy link
Member Author

Hiroshiba commented May 25, 2024

こちらの実装に関して、issue内容に書いてあることをそのまま実践すると破壊的変更になることに気づきました。
スタイルごとのアイコンがOptionalになってしまうためです。
(ボイスサンプルの方は依存してるサードパーティーが少ないかもだけど、スタイルごとのアイコンは依存している人がそこそこ多そう)

破壊的変更は抑えつつ、リソース内で共通アイコン画像を設定できるようにすれば良い気がします。

  1. 今まで
    • Styleごとにアイコンを返す
    • リソースでもスタイルごとにアイコン画像が必要
  2. 提案の形
    • Styleごとにアイコンを返す
    • リソースでは共通のアイコン画像を用意しても良い
  3. (すでに実装してしまったプルリクエストの形)
    • 共通アイコンを返し、スタイルごとにアイコンを返さないこともある(破壊的変更)
    • リソースでは共通のアイコン画像を用意しても良い

なのですでに実装してしまった #1052 の内容を変更し、↑の「2 提案の形」に合わせようかなと思ってます。

特にコメント等なければ2の流れで進めようと思います!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
機能向上 状態:実装 実装をおこなっている状態
Projects
None yet
3 participants