search

webmachinelearning / webnn-polyfill

🧠⚙️ Web Neural Network API polyfill based on TensorFlow.js
https://www.npmjs.com/package/@webmachinelearning/webnn-polyfill
Apache License 2.0
102 stars 18 forks source link

Skip the test case for ML when it runs on Node.js #90

Closed lionkunonly closed 3 years ago

lionkunonly commented 3 years ago

Skip the test case for ML when it runs on Node.js so that the error will be skipped for webnn-native. It is reasonable because the navigator does not exist in Node.js. The related issue is Webnn-native #610.

mingmingtasd commented 3 years ago

Skip the test case for ML when it runs on Node.js so that the error will be skipped for webnn-native. It is reasonable because the navigator does not exist in Node.js. The related issue is Webnn-native #610.

Thanks! Update commit id here and do gclient sync. @lionkunonly

huningxin commented 3 years ago

Please fix the lint error. You can reproduce it by

> npm run lint
lionkunonly commented 3 years ago

Please fix the lint error. You can reproduce it by

> npm run lint

I have fixed the lint error. Pls check it.

Skip the test case for ML when it runs on Node.js so that the error will be skipped for webnn-native. It is reasonable because the navigator does not exist in Node.js. The related issue is Webnn-native #610.

Thanks! Update commit id here and do gclient sync. @lionkunonly

Thanks for your suggestions. I will send a PR to the webnn-native with a new commit id when this PR is merged in webnn-polyfill.