Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when using tensorflow embegging. #36

Open
13bmartens opened this issue Apr 13, 2023 · 3 comments
Open

Error when using tensorflow embegging. #36

13bmartens opened this issue Apr 13, 2023 · 3 comments

Comments

@13bmartens
Copy link

Hi Team, did some experimentation and found this issue when trying different tensorflow based embedding approaches:

from grape.embedders import StructuredEmbeddingTensorFlow
embedding = StructuredEmbeddingTensorFlow().fit_transform(graph)

produces error:

ValueError: We have found an useless method in the class StubClass, implementing method Structured Embedding from library TensorFlow and task Node Embedding. It does not make sense to implement the `requires_positive_edge_weights` method when the `can_use_edge_weights` always returns False, as it is already handled in the root abstract model class.

Anything I am doing wrong?

Thanks!

@LucaCappelletti94
Copy link
Member

Hi @13bmartens, that is a misleading error that I need to fix. The correct error would be that some dependency is missing, and therefore it cannot load the proper model.

The short of it is that, given the model you are trying to load, most likely you do not have tensorflow installed, do let me know if that is the case.

@13bmartens
Copy link
Author

That was the initial reason, I installed tensorflow now and the error persists:

import tensorflow as tf
print(tf.version)

2.12.0

@LucaCappelletti94
Copy link
Member

Ok! Could you please join the discord server, and we can look into it there in a call?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants