Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@taicchoumsft
Copy link
Member

Description

Enable disambiguation support in Orchestrator across the board. This PR doesn't automatically add the OnChooseIntent trigger, the user needs to do this manually.

Also, there's a case of what should happen when say, QnA and Orchestrator are used in the same dialog. If CrossTrainedRecognizerSet fires a ChooseIntent with Orchestrator's top scoring intent also "ChooseIntent", then the user has to do some work to extract out Orchestrator's actual results.

Task Item

fixes #5747

Screenshots

@taicchoumsft taicchoumsft self-assigned this Apr 3, 2021
@taicchoumsft taicchoumsft changed the title feature: Enable the disambiguation across the board for Orchestrator feature: Enable disambiguation across the board for Orchestrator Apr 3, 2021
@taicchoumsft taicchoumsft changed the title feature: Enable disambiguation across the board for Orchestrator WIP: Enable disambiguation across the board for Orchestrator Apr 3, 2021
@coveralls
Copy link

coveralls commented Apr 3, 2021

Coverage Status

Coverage remained the same at 51.691% when pulling fbf5668 on tachou/orchDisambiguation into 1bff121 on main.

@taicchoumsft taicchoumsft changed the title WIP: Enable disambiguation across the board for Orchestrator feat: Enable disambiguation across the board for Orchestrator Apr 4, 2021
@boydc2014 boydc2014 marked this pull request as ready for review April 7, 2021 00:10
@boydc2014 boydc2014 merged commit f551a78 into main Apr 7, 2021
@boydc2014 boydc2014 deleted the tachou/orchDisambiguation branch April 7, 2021 04:50
@cwhitten cwhitten mentioned this pull request May 20, 2021
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Orchestrator: ChooseIntent (aka Duplicate Intent) Trigger doesn't work when using Orchestrator

4 participants