-
Notifications
You must be signed in to change notification settings - Fork 72
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
UTF-8 and Optionals #37
Comments
Maybe the Chinese symbols problem is caused by space. p = r'(?:\\s|\\b)+{}(?:\\s|\\b)+'.format(p) How about change it to this? p = r'(?:\\s|\\b)?{}(?:\\s|\\b)?'.format(p) But this may cause the English in word optionals. Sorry I am not familiar with RiveScript and this is only my thoughts. |
htdinh
pushed a commit
to htdinh/rivescript-python
that referenced
this issue
Sep 21, 2017
htdinh
pushed a commit
to htdinh/rivescript-python
that referenced
this issue
Sep 21, 2017
kirsle
added a commit
that referenced
this issue
Sep 21, 2017
#37 Fix unicode handling with optionals
Fixed by PR #108 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This might be related to rivescript-js#147. The following test case produces the problem:
The words with umlauts in them make the second trigger unmatchable.
In rivescript-js#147 I found Python wasn't affected by the Chinese symbols but may be having problems with these umlauts for some reason.
The text was updated successfully, but these errors were encountered: