-
Notifications
You must be signed in to change notification settings - Fork 206
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
コアの変更への追従 #385
Comments
ファイル名をコアの判定に使っていたことを完全に忘れていました・・・。 ファイル名統一の変更を加えると、過去バージョンのコアはもう使えなくなりそうな気がしています。 |
問題を整理したいと思ったので、現在の自分の認識と、考えうる対応策をまとめてみました。間違いや考え漏らし等ありましたらご指摘ください。 エンジンでのコア使用の流れ現状は、以下のような流れで
今回問題となるのは 3 の手順で、takana さんのご指摘の通り以下の2つの問題がありそうです:
コアのタイプの自動判定のサポート範囲をどうするかによって、自動判定方法に関して以下のように対応が変わってきそうです: libtorch 版コアの自動判定を引き続きサポートする場合
onnxruntime 版コアの自動判定のみをサポートする場合
|
ちょっと不確定かもしれませんが、dllなどがあるディレクトリ内に
|
なるほど、いいアイデアですね! 各条件を表にすると以下のような感じですね。
そして挙げていただいた以下の順に判定すると、確かに3種類のコアを判別できそうです。
|
なるほです!!! まとめありがとうございます!! とりあえず今までのコードも活用できそうで、かつ名前統一もできてライブラリをより簡単に利用できて良いなと感じました。 (製品版もビルドする準備があるので、適用され次第ビルドしてみようかなと思っています。) |
いいと思います! エンジン側の対応は、他に取り組んでいる方がいなければ自分がやってみようと思っています。 @takana-v さんは取り組んでいらっしゃいますか? |
取り組んでいないのでよろしくお願いします🙇 |
製品版のコアのビルドが完了しましたのでご報告です! |
内容
initialize関数の引数が変更されているのでCoreWrapperの
__init__
に変更が必要です。もしマージされた場合、コアのバージョン判別機構に変更が必要です。
Pros 良くなる点
Cons 悪くなる点
実現方法
VOICEVOXのバージョン
0.?.0
OSの種類/ディストリ/バージョン
その他
完全に忘れていたのでメモ代わりのIssueです。
もし途中まで取り組んでいる方がいらっしゃったらself-assignをお願いします。
The text was updated successfully, but these errors were encountered: