-
Notifications
You must be signed in to change notification settings - Fork 289
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
Simple Example uses non-existent PMI argument #122
Comments
Thanks for pointing this out and including Scattertext in the spaCy universe. I'm preparing to deprecate the
Regardless, I'll update the package to ensure the |
Ah, thanks for the info about the example! We've already merged the PR I linked to, but if you'd like to update the Universe entry we'd be happy to look at a PR any time. (That said, we're currently working on our website backend, so any updates in the immediate future won't go live for a bit.) |
Thanks for working on this package. I updating the entry in the spaCy Universe (explosion/spaCy#11937 (review)) and we noticed the sample here uses an argument that doesn't seem to work with the latest release.
scattertext/simple.py
Line 19 in 8ddff82
The text was updated successfully, but these errors were encountered: