-
-
Notifications
You must be signed in to change notification settings - Fork 334
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
Remove deprecated arg #11
Conversation
should also include #18 (comment) ? |
Can Someone please approve and Merge ? |
Works pretty well! Can you please merge this PR? Non-English Spacy modules require version >= 2.0... |
Is this solution merged ? The master doesn't seems compatible to latest Spacy version. |
Just ran in to this problem, duplicated this PR and it does fix the issue. Would be nice if it were merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks @laxatives
No description provided.