-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
CLN Fixes PendingDeprecationWarning in CountVectorizer #19299
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
CLN Fixes PendingDeprecationWarning in CountVectorizer #19299
Conversation
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.
Thanks for this PR. A further step to get rid of np.matrix
.
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.
LGTM.
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.
LGTM
…19299) Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
word n-grams not working for Devanagari script. Tokens are formed only until a modifier is found, which should not be the case. The default should be a whitespace.
Output: |
Reference Issues/PRs
Address a part of #12327
What does this implement/fix? Explain your changes.
Removes the use of
np.matrix
inCountVectorizer.inverse_transform