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.

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

implemantation of lookup tables for rasa_nlu #42

Closed
schwepmo opened this issue Oct 4, 2018 · 3 comments
Closed

implemantation of lookup tables for rasa_nlu #42

schwepmo opened this issue Oct 4, 2018 · 3 comments

Comments

@schwepmo
Copy link

schwepmo commented Oct 4, 2018

Hey,
with rasa_nlu 0.13.3 rasa introduced lookup tables, which have a similar notation
as synonyms.
Correct me if I'm wrong, but I don't think Chatito offers a way to specify lookup tables yet.
I think it would be a nice feature to add.
Cheers

@rodrigopivi
Copy link
Owner

rodrigopivi commented Oct 9, 2018

hi
This is something that will be implemented for the next version, atm im with some work load, so this may take some weeks, meanwhile, you can use the generated parser utility to get the AST object.

An example of this is at the parser tests https://github.com/rodrigopivi/Chatito/blob/master/src/tests/parser.spec.ts

@schwepmo
Copy link
Author

schwepmo commented Oct 9, 2018

hey @rodrigopivi,
yeah sure no hurries. I've got some freetime on my hands this month. I'll maybe
try to read into the code and open a PR. :)
cheers

@rodrigopivi
Copy link
Owner

did a quick research on this topic, adding lookup tables won't be considered ATM since most of the datasets generated by Chatito are non deterministic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants