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

Fix 346 #349

Merged
merged 11 commits into from
Dec 19, 2017
Merged

Fix 346 #349

merged 11 commits into from
Dec 19, 2017

Conversation

pminervini
Copy link
Collaborator

Now it's possible to install jack with or without GPU support, like this:

jack$ pip install -e .[tf]
[..]
jack$ pip install -e .[tf_gpu]
[..]

@codecov-io
Copy link

codecov-io commented Dec 19, 2017

Codecov Report

Merging #349 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c762855...19d52c6. Read the comment docs.

@dirkweissenborn
Copy link
Collaborator

spacy must be version 1.9! See #338 .

@pminervini
Copy link
Collaborator Author

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

@dirkweissenborn
Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants