search

nikhilk / node-tensorflow

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

Revise Tensor/Graph APIs to be simple script types #26

Closed nikhilk closed 6 years ago

nikhilk commented 6 years ago

Currently Tensor and Graph wrap a native handle, and must be explicitly deleted. This is sub-optimal.

Instead, we could make Tensor and Graph be vanilla script objects, and internally build associated TF objects within Session, either when creating a Session, or when calling session.run.

nikhilk commented 6 years ago

Addressed in faa2a6d32317a59655643c73b1faeb3f01a217eb and included as part of 0.7.0.

Updated samples show simplifications.