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

Vocabulary module does not have an option for timeout #2

Closed
natefanaro opened this issue Dec 26, 2017 · 1 comment
Closed

Vocabulary module does not have an option for timeout #2

natefanaro opened this issue Dec 26, 2017 · 1 comment
Assignees
Labels

Comments

@natefanaro
Copy link
Contributor

https://github.com/blissend/hqtrivia-automation/blob/master/hqtrivia-automation.py#L64

timeout is not implemented in tasdikrahman/vocabulary I noticed the issue you raised here tasdikrahman/vocabulary#60 and am guessing there is a custom version of vocabulary floating around (but wasn't publicly forked)

For now removing timeout works, but the script is too slow without it.

@blissend
Copy link
Owner

blissend commented Dec 26, 2017

Yes, I just ended up editing my local vocabulary for timeouts. Then I made it use nltk (local dictionary) first before using vocabulary (online dictionary). In the next update each lookup will be in its own process/thread rather than each answer to speed things up even more.

I will make sure to remove what you pointed out until vocabulary has this in officially. Should have it out by tomorrow.

@blissend blissend self-assigned this Dec 26, 2017
@blissend blissend added the bug label Dec 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants