waylonflinn / weblas

GPU Powered BLAS for Browsers :gem:
MIT License
702 stars 43 forks source link

lib/test.js should not be part of main package #34

Closed transcranial closed 7 years ago

transcranial commented 7 years ago

Importing weblas from npm package gives the following error, as arrayloader is a devDependency:

ERROR in ./~/weblas/lib/test.js
Module not found: Error: Can't resolve 'arrayloader' in node_modules/weblas/lib'
 @ ./~/weblas/lib/test.js 2:10-32
 @ ./~/weblas/index.js
 @ ./src/Tensor.js
 @ ./src/index.js
 @ multi main
waylonflinn commented 7 years ago

Thanks Leon.

This should already be fixed in master (though the commit message doesn't mention it). I'm publishing a new version to npm right now.

waylonflinn commented 7 years ago

Can I close this?

transcranial commented 7 years ago

All good, thanks!