-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Should "/" not be a separate token and cause a split? #891
Comments
Thanks – and that's weird, I totally would have thought that this was already split correctly. Just added a regression test and played around with this quickly – it might be as simply as just adding the But we need to make sure to test this properly and make sure it doesn't break any of the tokenizers that rely on the default punctuation. |
All worked as expected and the fix will be included in the next point release. The likely reason why |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Perhaps you're thinking of more common cases where it shouldn't be split? I'd almost think you should make "/" to be a
conj
.The text was updated successfully, but these errors were encountered: