-
Notifications
You must be signed in to change notification settings - Fork 157
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
Add LUIS Adapter #61
Comments
Thanks for the request @cswendrowski, you are mostly welcome to open a PR for review. Keeping an eye and supporting all NLU framework data formats takes more time than i've been able to provide so far... open source contributions are mostly welcome. |
@rodrigopivi can you point me to the other files I will need to update to propagate the adapter changes through the rest of the codebase? |
@cswendrowski you can start by creating a new adapter and using it with the CLI tool by importing it from file If that is done, then the IDE integration is just a react app at web directory, i can update that later given that the first steps are done. |
Done! Luis adapter is now available since v2.2.0 |
Request
LUIS has an endpoint to batch add Labeled Utterances (https://westus.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/5890b47c39e2bb052c5b9c09). LUIS uses the same data format to allow batch testing via their UI (https://docs.microsoft.com/en-us/azure/cognitive-services/LUIS/luis-how-to-batch-test)
It would be convenient to generate out Train and Test data in the LUIS format. LUIS supports synonyms via Phrase Lists and List Entities, but these are separate endpoints which would require different file outputs and therefore would be far more complicated to support.
Data
LUIS's data format is almost identical to the core of RASA's
RASA schema:
LUIS schema:
Current thoughts on implementation
After taking a look at how the RASA adapter is implemented it seems like a pretty simple adjustment to make it output LUIS's format instead. I'm not sure how I would go about hooking that up into the rest of the system however
The text was updated successfully, but these errors were encountered: