Skip to content
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.

Sign up for GitHub

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

Jump to bottom

RegEx intent provider should support mapping synonyms #261

Open
gdaniel opened this issue Nov 29, 2019 · 0 comments
Open

RegEx intent provider should support mapping synonyms #261

gdaniel opened this issue Nov 29, 2019 · 0 comments
Assignees
@gdaniel
Labels
Enhancement New feature or request Low Priority
Milestone
Awaiting Integration

Comments

@gdaniel
Copy link
Member

gdaniel commented Nov 29, 2019

Supporting synonyms would also allow to match mapping values in a case insensitive way (see #259)
We need synonyms for this because

  • Matched mappings should return their reference value (so we cannot just accept the input in a case insensitive way as another mapping value, otherwise we are not returning the same mapping value)
  • Case insensitive regex derived from the base value could be transparently added as a synonym
@gdaniel gdaniel added the Enhancement New feature or request label Nov 29, 2019
@gdaniel gdaniel added this to the Awaiting Integration milestone Nov 29, 2019
@gdaniel gdaniel self-assigned this Nov 29, 2019
gdaniel added a commit that referenced this issue Nov 29, 2019
@gdaniel
RegEx intent recognition is now case insensitive (fix #259)
7bbd21f 
The training sentences are case insensitive, but not mapping entities (refs #261). To do this we need support for synonyms in the RegExIntentRecognitionProvider
@gdaniel gdaniel mentioned this issue Nov 29, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gdaniel gdaniel

Labels
Enhancement New feature or request Low Priority
Projects
None yet
Milestone
Awaiting Integration
Development

No branches or pull requests
2 participants
@jcabot @gdaniel