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

ソング未対応のコアを読み込めなくなっている #1087

Closed
Hiroshiba opened this issue Feb 27, 2024 · 5 comments · Fixed by #1110
Closed

ソング未対応のコアを読み込めなくなっている #1087

Hiroshiba opened this issue Feb 27, 2024 · 5 comments · Fixed by #1110

Comments

@Hiroshiba
Copy link
Member

不具合の内容

エンジンは過去のコアに関してある程度の互換性を保つように設計されています。
しかしソング実装のタイミングで、ソング未実装のコアライブラリを読み込めないようになっていました。
とりあえず現状のまとめをissueにしてみました。

現象・ログ

0.14.6などの過去のコアを読み込もうとするとエラーになる

期待動作

0.14.6などの過去のコアも読み込める

その他

関連

@Hiroshiba
Copy link
Member Author

@PickledChair さん @takana-v さん

すみません、もしよかったらお力お借りできないでしょうか・・・ 🙇
PythonのCDLL周りに詳しくなく・・・。

ちなみに過去のコアを配置できる場所は以前から変わらずユーザーディレクトリとかのままです!
https://github.com/VOICEVOX/voicevox_engine#%E3%83%A6%E3%83%BC%E3%82%B6%E3%83%BC%E3%83%87%E3%82%A3%E3%83%AC%E3%82%AF%E3%83%88%E3%83%AA%E3%81%AB%E9%85%8D%E7%BD%AE%E3%81%99%E3%82%8B

(自分の把握漏れなのにお願いしてしまって申し訳ないです 🙇 )

@takana-v
Copy link
Member

まとまった時間が取れるのが3月中旬以降になってしまうため、もし急ぎであれば他の方に修正をお願いしたいです🙇
(最近のコアの変更などが追えてないので、仕様を把握するのに時間が掛かりそうです...)

@qryxip
Copy link
Member

qryxip commented Feb 28, 2024

とりあえずメソッドの存在の判別は、hasattrでできると思います。

python
Python 3.8.15 (default, Nov  6 2022, 22:50:15)
[GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from ctypes import CDLL
>>> core = CDLL("./libvoicevox_core.so")
>>> "initialize" in dir(core)
False
>>> hasattr(core, "initialize")
True
>>> "initialize" in dir(core)
True
>>> hasattr(core, "存在しないメソッド")
False

@PickledChair
Copy link
Member

明後日くらいから時間ができるので、実装を担当できます

@qryxip

とりあえずメソッドの存在の判別は、hasattrでできると思います。

その方法で行こうと思います👍

@Hiroshiba
Copy link
Member Author

@PickledChair ありがとうございます!!
もし難しくなったりしたら遠慮なく言っていただけると!! 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants