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

Lsi distributed fail #807

Closed
carlos386386 opened this issue Jul 27, 2016 · 12 comments · Fixed by #968
Closed

Lsi distributed fail #807

carlos386386 opened this issue Jul 27, 2016 · 12 comments · Fixed by #968
Assignees
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix

Comments

@carlos386386
Copy link

Hi,
I've got a problem with the lsi distributed. When i executed the example:

https://radimrehurek.com/gensim/dist_lsi.html

First configure the server (enviroment variables), then i run the server, worker and dispatcher.

And all without errros. But when i executed the code. I have this fail:
image

Why does this happens? How can i solve?

Thank you in advance.

@piskvorky piskvorky added the bug Issue described a bug label Jul 29, 2016
@piskvorky
Copy link
Owner

piskvorky commented Jul 29, 2016

Hello @carlos386386 -- what version of pyro & gensim? (what's your output of pip freeze)

See CONTRIBUTING.md for reporting issues.

@carlos386386
Copy link
Author

Pyro4==4.46
gensim==0.12,1

@menshikh-iv
Copy link
Contributor

I think adding @exposure decorator to methods that called remotely fix this problem

@carlos386386
Copy link
Author

carlos386386 commented Aug 1, 2016

I tried to with this method:

image

With the decorator and still fails

@menshikh-iv
Copy link
Contributor

@carlos386386 Need to decorate methods that call remotely (some inner methods in lsi_worker) like this

@carlos386386
Copy link
Author

I have modify the file lsi_worker??

@menshikh-iv
Copy link
Contributor

@carlos386386 yes

@piskvorky
Copy link
Owner

Must be some recent, breaking change in Pyro API.

Your Pyro version (4.46) doesn't indicate any major API change though. So not sure what's happened -- will need deeper investigation.

@tmylk
Copy link
Contributor

tmylk commented Sep 18, 2016

@menshikh-iv Are you suggesting that we should add annotations to lsi_worker same way as lda_worker? May I kindly ask for a PR for this?

@menshikh-iv
Copy link
Contributor

@tmylk Yes, but a little later, right now I have no time :(

@tmylk tmylk added the difficulty easy Easy issue: required small fix label Sep 25, 2016
markroxor added a commit to markroxor/gensim that referenced this issue Oct 7, 2016
markroxor added a commit to markroxor/gensim that referenced this issue Oct 7, 2016
@markroxor
Copy link
Contributor

@tmylk should this be closed?

@tmylk
Copy link
Contributor

tmylk commented Oct 20, 2016

@markroxor the annotations are still missing in lsi_worker

mon95 pushed a commit to mon95/gensim that referenced this issue Nov 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants