-
Notifications
You must be signed in to change notification settings - Fork 205
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
[project-s] ハミングスタイルはTTSできないので、全スタイルでTTSできることを前提にしたアプリがエラーになる #1027
Comments
@y-chan お手空きの際にご意見いただけると。。。 🙇 |
1のアイデアに近いですが、ハミング用スタイルは別のURI('/sing/speakers'等)で提供し、既存の'/speakers'はそのままにしておくのはどうでしょうか? |
@Segu-g 確かにと思ったのですが、ちょっと記憶を掘り返して何でまとめたか思い出しました!! |
@Hiroshiba なるほど!経緯が理解できました。 ただ、現状 |
あれ、ここに認識の違いがあるかもです! 正確に言うと |
こちらですが、とりあえず |
不具合の内容
project-sで不都合を思いついたのでメモです。
ハミングスタイルは他のスタイルと同様、style idと名前を持ちます。
今までのスタイルとの区別は新しく追加される
type
で識別する予定です。エディタ側ではこのtypeを用いてフィルタリングを行い、talkスタイルだけをTTS欄に、hummingスタイルは歌手として表示される形を考えています。
問題はサードパーティアプリで、style typeが新たに追加されて、かつTTS非対応なスタイルが増えることを知らないはずです。
なのでTTS一覧に普通にハミングが表示されたり、ハミングとわからない(nameがtalkのと一緒なので)形になりそうです。
アップデート準備期間が長ければ問題はないかもしれませんが、今回はいきなりの変更になるので、できればなんとかしたいです。
思いつく手はいくつかあります。
個人的には1が便利そうに感じてます。
やるならたぶん
/speakers
と/speaker_info
の両方につける形かなと。style_type_filter: list["talk" | "humming" | "sing_teacher"] | None
みたいな引数を追加して、デフォルトは["talk"]
とか・・・。その他
なんかまだ見落としてることありそうな気がしないでもない。。。
The text was updated successfully, but these errors were encountered: