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

ValueError when trying to use external embedder like in pykeen and karateClub #25

Closed
gsanou opened this issue Nov 7, 2022 · 3 comments

Comments

@gsanou
Copy link

gsanou commented Nov 7, 2022

Hello,
Thanks you for your amaeing work, i'm a phD student working on the embeddings of biomedical data particularly in immunogenetics, and currently i'm comparing tools to embed data. I found your works very interesting. I got some issues when i try to use external model from pykeen and karateclub. i got this message :
ValueError: We have found an useless method in the class StubClass, implementing method HolE from library PyKEEN 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.

Also for the vizualisation, when i did ```  from grape import GraphVisualizer
visualizer = GraphVisualizer(kg.remove_disconnected_nodes())
visualizer.fit_and_plot_all(embedding)

I got this warning without no visualisation:  FutureWarning: The parameter `square_distances` has not effect and will be removed in version 1.3.
Thank you in advance for your answer
Gaoussou
@LucaCappelletti94
Copy link
Member

Hello @gsanou, please attach the code you used.

Moreover, be advised that the error you are getting is because you do not have installed PyKEEN; admittedly, it should be handled better, and I'll look into it.

Secondly, the FutureWarning error you see is a sklearn one, independent from 🍇.

@LucaCappelletti94
Copy link
Member

LucaCappelletti94 commented Nov 7, 2022

I have tested now on COLAB the following:

First, I install 🍇 and PyKEEN:

! pip install grape pykeen -Uq

and then I run the one-liner:

from grape import GraphVisualizer

GraphVisualizer("Cora", repository="linqs")\
    .fit_and_plot_all("HolE")

And you get:

image

I hope everything is clear.

@gsanou
Copy link
Author

gsanou commented Nov 7, 2022

Thank you @LucaCappelletti94 . I think it was pykeen problem. i will close it.
Gaoussou

@gsanou gsanou closed this as completed Nov 7, 2022
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