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

feat: LuisAdaptiveRecognizer and test mocks #3275

Merged
merged 37 commits into from
Feb 8, 2021
Merged

feat: LuisAdaptiveRecognizer and test mocks #3275

merged 37 commits into from
Feb 8, 2021

Conversation

chon219
Copy link
Member

@chon219 chon219 commented Feb 5, 2021

Fixes #3036
Fixes #3002
Fixes #3018

Description

Updated LuisAdaptiveRecognizer to use expression based configurations.

Specific Changes

  • Updated LuisAdaptiveRecognizer to use expression based configurations.
  • Added support for externalEntityRecognizer in LuisAdaptiveRecognizer
  • Updated MockHttpRequestMiddleware to support mocking LUIS requests
  • Updated LuisAdaptiveRecognizer to use v3 endpoint by default

Testing

Ported tests from C# repo.

@chon219 chon219 added Area: Adaptive The issue is related to Adaptive dialogs Area: AI-LUIS The issue is related to LUIS labels Feb 5, 2021
@chon219 chon219 added this to the R12 milestone Feb 5, 2021
@chon219 chon219 requested review from a team as code owners February 5, 2021 10:39
@chon219 chon219 requested a review from a team as a code owner February 5, 2021 10:39
@coveralls
Copy link

coveralls commented Feb 7, 2021

@joshgummersall joshgummersall merged commit d5adca1 into main Feb 8, 2021
@joshgummersall joshgummersall deleted the zim/luis branch February 8, 2021 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Adaptive The issue is related to Adaptive dialogs Area: AI-LUIS The issue is related to LUIS
Projects
None yet
3 participants