iesl / box-embeddings

Box Embeddings as Modules
https://www.iesl.cs.umass.edu/box-embeddings
Apache License 2.0
100 stars 10 forks source link

Tf.Tensor is not an instance of Tf.Variable #42

Closed tejas4888 closed 3 years ago

tejas4888 commented 3 years ago

isinstance(tf.Variable([1]), tf.Tensor) returns False but returns True for tf.constant([1]). The variable implementation will be required for computing gradients as constants cannot be modified in tensorflow. Currently the code is functional for the tf.constant([1]) implementation