search

transcranial / keras-js

Run Keras models in the browser, with GPU support using WebGL
https://transcranial.github.io/keras-js
MIT License
4.96k stars 501 forks source link

Changing main to point to built files in dist/keras.js #55

Closed jefffriesen closed 7 years ago

jefffriesen commented 7 years ago

I think maybe there was a mistake with a global find & replace here: https://github.com/transcranial/keras-js/commit/fb6527466490ecf6a978f97709ee8c5fcc39175a

main in package.json was pointing to the uncompiled files in lib/index.js. So when this package was installed with npm and required, it required compilation of everything in the src file. For example, it required babel-polyfill, axios, glsl files and other things that aren't really part of this library.

This change tells applications who require keras-js to load the already compiled files.

transcranial commented 7 years ago

Indeed. Sorry about the confusion.