-
-
Notifications
You must be signed in to change notification settings - Fork 190
ort-web
#450
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
base: main
Are you sure you want to change the base?
ort-web
#450
Conversation
|
@decahedron1 this is awesome, lmk if you need help testing! |
|
This should be ready for testing now. Make sure to read this before using: Lines 1 to 78 in 15bb0e4
For reference, a very very very minimal sample program can be found at https://github.com/decahedron1/ort-web-sample. It can be built with Please reach out here if you have any questions/concerns! |
|
@decahedron1 sounds good. I'll do it later this week. |
| * @property {Record<'main' | 'wrapper' | 'binary', string>} integrities | ||
| */ | ||
|
|
||
| const DIST_BASE = 'https://cdn.pyke.io/0/pyke:ort-rs/web@1.23.0/'; |
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.
In relation to the backends/web/lib.rs doc comment:
Make sure
cdn.pyke.iois accessible via CORS if you have that configured.
ort-webdynamically fetches the required scripts & WASM binary from this domain at runtime.
Is there any intention to support customizing this for self-hosted / internal distribution where third-party CDNs may not be available?
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.
Yep! I just wanted to keep things simple for now.
|
Awesome! I try to give it a spin soon as well 😎 |
|
Damn this is awesome! I was using my own bindings and a small trait based wrapper for bridging native and web apis into a single cross-platform codebase. Time to throw mine out once this is stabilised 🚀 |
No description provided.