nikhilk / node-tensorflow

Node.js + TensorFlow
Apache License 2.0
587 stars 59 forks source link

Free up one of the two npm packages #6

Closed ZECTBynmo closed 6 years ago

ZECTBynmo commented 8 years ago

Why does this repo need to be published under tensorflow and node-tensorflow? Seems like it'd be better to allow for competing implementations (especially since development seems to have stalled here).

https://www.npmjs.com/package/node-tensorflow https://www.npmjs.com/package/tensorflow

goatandsheep commented 6 years ago

I think node-tensorflow is dead, but tensorflow is being updated

ralphtheninja commented 6 years ago

@nikhilk I think you hould consider removing the node- prefix from the repository name since you are publishing it as tensorflow anyway.

nikhilk commented 6 years ago

I added node- to the repo name as it would conflict if you had a branch of the tensorflow repo (which I did at some point for a fork containing blaze targets to build tensorflow libs).

However require(tensorflow) works better in code... hence that package name.

goatandsheep commented 6 years ago

Perhaps a better idea is to change the readme in the description of npm/node-tensorflow to forward to npm/tensorflow