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

[js/web] fix a few package consuming problems #18109

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Oct 26, 2023

Description

This PR tries to fix a part of the NPM package consuming problems for onnxruntime-web (ES module) as described in #10913:

  • reduce the package size to fit the 150MB restriction in jsdelivr, by removing dev build targets for uncommon exports
  • add default export to support import ort from 'onnxruntime-web'; (currently only support import * as ort from 'onnxruntime-web';

@guschmue guschmue merged commit 9bba990 into main Oct 30, 2023
54 of 57 checks passed
@guschmue guschmue deleted the fs-eire/fix-npm-package-distrib branch October 30, 2023 15:11
kleiti pushed a commit to kleiti/onnxruntime that referenced this pull request Mar 22, 2024
### Description
This PR tries to fix a part of the NPM package consuming problems for
onnxruntime-web (ES module) as described in microsoft#10913:

- reduce the package size to fit the 150MB restriction in jsdelivr, by
removing dev build targets for uncommon exports
- add default export to support `import ort from 'onnxruntime-web';`
(currently only support `import * as ort from 'onnxruntime-web';`
siweic0 pushed a commit to siweic0/onnxruntime-web that referenced this pull request May 9, 2024
### Description
This PR tries to fix a part of the NPM package consuming problems for
onnxruntime-web (ES module) as described in microsoft#10913:

- reduce the package size to fit the 150MB restriction in jsdelivr, by
removing dev build targets for uncommon exports
- add default export to support `import ort from 'onnxruntime-web';`
(currently only support `import * as ort from 'onnxruntime-web';`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants