-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Error when running tfjs-automl in web worker due to old tfjs-core dependency #8186
Comments
Hi, @PrafulB Thanks for bringing this issue to our attention! I understand that the current dependency on Upgrading the dependency in We appreciate your willingness to contribute and submit a pull request to address this issue. Our team will thoroughly review your PR and take appropriate action based on their assessment. Thank you again for your valuable contribution. |
This issue has been marked stale because it has no recent activity since 7 days. It will be closed if no further activity occurs. Thank you. |
This issue was closed due to lack of activity after being marked stale for past 7 days. |
Updated the version of the tfjs-core dependency in tfjs-automl to resolve the ReferenceError when run from within a web worker.
Apologies for not noticing your response earlier, @gaikwadrahul8 . I've created a PR ( #8256 ) that updates the version of the tfjs-core dependency. All tests run successfully for me. Appreciate the help! |
Describe the current behavior
tfjs-automl depends on tfjs-core v3.9.0 , which causes a ReferenceError when run inside a Web Worker. This is documented in issue #6922 .
Describe the expected behavior
Upgrading the dependency in the package.json to tfjs-core v4.0.0+ should resolve this issue. Tried setting the dependency to tfjs-core v4.9.0, and all tests complete successfully.
Happy to submit a PR with the above change if it helps.
The text was updated successfully, but these errors were encountered: