-
Notifications
You must be signed in to change notification settings - Fork 82
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
Fix 346 #349
Fix 346 #349
Conversation
Codecov Report
@@ Coverage Diff @@
## master #349 +/- ##
=======================================
Coverage 48.82% 48.82%
=======================================
Files 88 88
Lines 5124 5124
=======================================
Hits 2502 2502
Misses 2622 2622 Continue to review full report at Codecov.
|
spacy must be version 1.9! See #338 . |
Oh now I get it - using an older version of a package because the newer is much slower is very lame, I'll dig into SpaCy's issues for finding a solution to this later |
I did that. They changed the models and they are just super slow now, see explosion/spaCy#1508 . For our very primitive preprocessing I prefer speed. Anyway, in our input modules spacy is never really used yet. I have a branch however, where I use it. So I really suggest keeping it until the newer version is as fast. |
Now it's possible to install
jack
with or without GPU support, like this: