-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
If you use Vite, refer to my configuration. #128
Comments
Thanks. I have another problem though, the library seems to be trying to load some files (ort-wasm-simd.wasm) from the current path instead of from root, therefore it's failing. ie. Do you have this problem? Solved, had to add:
|
{
src: 'node_modules/@ricky0123/vad-web/dist/vad.worklet.bundle.min.js',
dest: './'
},
{
src: 'node_modules/@ricky0123/vad-web/dist/silero_vad.onnx',
dest: './'
},
{
src: 'node_modules/onnxruntime-web/dist/*.wasm',
dest: './'
},
+ {
+ src: 'node_modules/onnxruntime-web/dist/ort-wasm-simd-threaded.mjs',
+ dest: './'
+ } For my instance I had to add the following and if you're in a workspace / mono repo make sure you're pathing correctly to the node_modules folder You may need to path dynamically or use absolute path |
Hey, FYI there is now an option called |
Still broken. I fell this pacakge is not easy to use with vite by every version. First, without local files config:
![]() Then I tried to copy the static files mentioned by doc, but the doc missed
However when I first tried the lib by html script weeks ago, It worked very well! The impression is awesome! Or ... should the repo change from webpack to vite? Since it's so popular now. |
Thanks to this thread I get it to work using vite.js and React : vite.config.js
React component
Hope this can help. |
Note that the latest release changed the dependency on @ricky0123 Any particular reason you made this change? I'm not sure but I suspect it's the reason for #88. |
Hi @matthewgertner it was actually more because of this one #161 and also this issue with onnxruntime |
Hey @ricky0123, I was just theorizing that the problem reported in #88 is due to use of an older version of Anyway I'm filtering out the warnings right now so (while it pains me to override |
Yeah, tbh the warnings are lower priority than some other pending improvements. But yeah, if the nextjs issue is fixed I'll upgrade onnxruntime. Also, I have a vague memory that someone figured out how to disable the warnings using our |
No worries, it's not an issue for me at all besides occasionally waking up in a cold sweat after a terrifying nightmare involving monkey-patching My main point was that readers of this thread should be aware that the latest code uses a version of ONNX that doesn't include the |
vite.config.js
in your code
The text was updated successfully, but these errors were encountered: