-
Notifications
You must be signed in to change notification settings - Fork 120
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
feat: metasにsupported_features
だけ追加
#900
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Segu <51497552+Segu-g@users.noreply.github.com> Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
モーフィング周りはかなり考えることがあって時間がかかるので、優先度的にレビューは後回し気味でも問題なさそうでしょうか…?🙇 現状のエンジン側の仕様を整理し、コアも設計から再確認していく形になりそうです。 |
#713 は後回し(来年)でもよいです。
このPRで考えることがあるとすれば、
一応現れることは現れますね。 |
あ〜どこにどう出すか、なんで名前にするか等いったん考えたいですね…! |
ただしJSONからデシリアライズする時は C APIから出すときとC API以外でJSONにシリアライズするときは、 こうすればプライベートっぽくはなるはず。そうすれば後はVVMのmetas.jsonかmanifest.jsonに入れるかだけ考えればよいと思います。 |
supported_featuresという名前にするのか、そもそもどこに情報を格納するのかなどが未定だけど、何も案内せずしれっと何も入ってないプロパティを足すということですよね。 そこまでしなくても、いろいろ決まった後に追加でも良いのかも、とちょっと思いました! |
内容
#713 の前に、metasに
supported_features
だけ入れる。以下のPRを参考にコードを書いた。
SpeakerSupporPermitedSynthesisMorphing
voicevox_engine#581permited
permitted
voicevox_engine#582@Hiroshiba と以下の1名の許諾のもと、 #874 にのっとりMITライセンスとしてライセンスする。
関連 Issue
Refs: #874 (comment)
その他