-
Notifications
You must be signed in to change notification settings - Fork 44
error while running how_to_use_ast2vec.ipynb #120
Comments
@monperrus, thank you for your report! It was a little outdated version of notebook since we update modelforge and move log-related functions there. I fix it and you will be able to run it when @vmarkovtsev merge PR. Also, be sure that you put some real repositories in |
Super, thanks!
Also, be sure that you put some real repositories in |py_repos| list.
What's this list? I'm not able find anything about it in the README.
|
@monperrus just look to the second cell in notebook you are trying to run: https://github.com/src-d/ml/blob/master/doc/how_to_use_ast2vec.ipynb There is some template value, that says that you need to put the real one. # repositories to use
py_repos = ['list', 'of', 'repositories', 'from', 'github', 'or', 'local', 'files', 'with', 'repositories'] |
Thanks!
Do you I have to put complete HTTP github urls or only local folder names? Do I need to clone then
myself before or are they cloned automatically?
|
The URLs must be cloneable with |
Thanks. After setting the list, there is a new error. Increasing the timeout does not help, and the timeout occurs after a long period of CPU idleness. Waiting for network?
|
And with a smaller repo, another error. I had a look at the code but the error is not obvious.
|
@monperrus Thank you very much for your patience and testing! @zurk Please investigate this. @EgorBu Please suffer too. I think we should prepare a dockerized notebook which is guaranteed to be reproducible, with needed bblfsh inside and the rest of the quirks. I will have a personal look tomorrow. Let me apologize for this. I have extensively tested the command line apps but apparently did not pay enough attention to the notebook. Let me suggest to switch to the command line. @zurk should provide the commands tomorrow. |
on my way. |
With the second repository, everything is clear. FYI, for new versions you can check out it in babelfish documentation. Also, Ruby and Bash will be available soon. P.S.: Now we are working on a big ast2vec update. Also, it will be renamed to sourced ml. It will use new babelfish and our new and awesome https://github.com/src-d/engine. And please give me more time to investigate the first error. |
About this repo: https://github.com/INRIA/spoon It is encoding related issue because of this file: Continue to investigate. |
So, it is not a real problem. Because ast2vec just ignore such files. I found out that everything is working for me for https://github.com/INRIA/spoon when I run it using `jupyter notebook. But it takes 20 mins (extremely long for 1200 java files). Maybe it can be related to fresh Babelfish issue bblfsh/bblfshd#130. Not sure. There will be PR soon with some README changes and new docker container for the notebook. @vmarkovtsev mentioned it. |
Sorry for long response. |
I've installed ml
pip3 install -r requirements.txt
Now I'm executing the example:
jupyter-nbconvert --execute how_to_use_ast2vec.ipynb
But I get an error. Any idea to fix it?
The text was updated successfully, but these errors were encountered: