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

Windowsデフォルトのonnxruntimeが使われた時に気付けるようにしたい #644

Closed
Hiroshiba opened this issue Oct 13, 2023 · 0 comments · Fixed by #802
Closed
Labels
機能向上 要議論 実行する前に議論が必要そうなもの

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Oct 13, 2023

内容

Windowsにはデフォルトでonnxruntime.dllがインストールされていることがあります。
コアを使う際にVOICEVOXが使いたいonnxruntime.dllがなかった場合、上記dllが使われてpanicになります(バージョンが古いため)。

(Python版を動かした例)
The given version [14] is not supported, only version 1 to 10 is supported in this build.
thread '<unnamed>' panicked at C:\Users\runneradmin\.cargo\git\checkouts\onnxruntime-rs-6a272c20db5f1ac0\ebb9dcb\onnxruntime\src\lib.rs:203:5:
assertion `left != right` failed: Can not get ort api ptr.
  left: 0x0
 right: 0x0
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Traceback (most recent call last):
  File "C:\Users\hihok\Github\voicevox_core\example\python\run.py", line 104, in <module>
    asyncio.run(main())
  File "C:\Users\hihok\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\Users\hihok\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\hihok\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "C:\Users\hihok\Github\voicevox_core\example\python\run.py", line 35, in main
    logger.debug("%s", f"{voicevox_core.supported_devices()=}")
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pyo3_runtime.PanicException: assertion `left != right` failed: Can not get ort api ptr.
  left: 0x0
 right: 0x0

この問題は気づきづらいので、問題に着づけるようにしたり、あるいは避けたいかもしれません。
多分今思えばこちらのissueも同じ要因だった気がします。

例えばonnxruntime.dllの名前をvoicevox_onnnxruntime.dllなどに改名すると回避できるかも?

Pros 良くなる点

初学者が陥りやすいエラーを回避できる。

Cons 悪くなる点

いろいろややこしいかも。
エンジンの方も変更しないといけないかも。

その他

要不要含めて要議論かもです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
機能向上 要議論 実行する前に議論が必要そうなもの
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants