-
Notifications
You must be signed in to change notification settings - Fork 468
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
Add image classification web demo with WebGPU, CPU backends #840
Conversation
Hi @antimora |
I am not sure why this error is showing but I re-did the index.js import to use an old fashion style import. When you have a chance, please give a try again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing demo, great job! I have a few comments, but nothing very important. Let me know what you think, but we can probably merge this quite soon!
When I play around for a while, changing backend then selecting an image several times, I always end up with an error:
|
Strangely I get this error too on one of my Chrome profiles, but on the other it works. Don't know why. |
Yeah I came across some strange errors too when I immediately change and select image. I have a feeling there is a racing condition. Initially when I implemented switch backend and immediately run inference, I had similar errors. This is something to look into this. |
Pull Request Template
Checklist
run-checks
script has been executed.Changes
Added a new web demo example for classifying images using ndarray, candle and wgpu backends.
Testing
Tested locally in the browser. See screenshots:
WebGPU backed:
Ndarray:
Candle: