search

DeepGraphLearning / graphvite

GraphVite: A General and High-performance Graph Embedding System
https://graphvite.io
Apache License 2.0
1.22k stars 151 forks source link

largevis command line error #14

Closed PJthunder closed 5 years ago

PJthunder commented 5 years ago

I just have a try of the command line for the Large viz. But the command graphvite visualize filename actually returns an error: "TypeError: load() takes 1 positional argument but 2 were given". Is there something wrong with the arguments loading for the command line?

KiddoZhu commented 5 years ago

Thanks for pointing out this bug. You can modify this line to

app.load(vectors=vectors, perplexity=args.perplexity)

It should work now.

KiddoZhu commented 5 years ago

Fixed. Will update it to conda.

PJthunder commented 5 years ago

Thanks for your efforts. Just update it through conda. Not sure whether there's is anything wrong or not with my environment setup. The same error still exists.

The graphvite version is still 0.1.0 on my side. Not sure whether it is updated on conda side already or not, since this page shows the last modified on 08-08-2019.

KiddoZhu commented 5 years ago

Sorry I have some trouble with conda compilation now. The repo is fixed. You'd better manually modify that file. It could be found in /your/conda/env/lib/python*.*/site-packages/graphvite/cmd.py.

Please understand conda compilation for all versions is time-consuming, so the conda packages usually have some lag.

PJthunder commented 5 years ago

No problem. Just edit that locally. It looks working now.

KiddoZhu commented 5 years ago

Updated to conda.