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.

Sign up for GitHub

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

Jump to bottom

Synthesisのload_all_modelsを廃止する #582

Closed
Tracked by #545
Hiroshiba opened this issue Aug 23, 2023 · 0 comments · Fixed by #587
Closed
Tracked by #545

Synthesisのload_all_modelsを廃止する #582

Hiroshiba opened this issue Aug 23, 2023 · 0 comments · Fixed by #587
Labels
機能向上

Comments

@Hiroshiba
Copy link
Member

内容

Synthesisのload_all_modelsを廃止する提案です。
全モデルのロード（全onnxruntimeセッションの作成）は昔のAPIにあったからサポートしていましたが、特にSynthesisの場合は各々のVVMをロードする時にonnxruntimeセッションを用意するか決められるため（たしか。）、この設定があまり使われない気がしました。

であればややこしさ軽減のためにもなくしてしまっても良さそうに思います。
この課題はSynthesisのload_all_modelsがなくなるか、なくさない結論が出れば完了だと思います。

Pros 良くなる点

APIがよりシンプルになる。

Cons 悪くなる点

実現方法

たぶん引数を消すだけ

その他

Discordのやり取りこの辺り
https://discord.com/channels/879570910208733277/893889888208977960/1139186983437094932
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
機能向上
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

load_all_modelsを廃止する qryxip/voicevox_core
1 participant
@Hiroshiba