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

Orchestrator UX integration #4390

Closed
boydc2014 opened this issue Oct 15, 2020 · 1 comment · Fixed by #4867
Closed

Orchestrator UX integration #4390

boydc2014 opened this issue Oct 15, 2020 · 1 comment · Fixed by #4867
Assignees
Labels
Area: Orchestrator P1 Painful if we don't fix, won't block releasing Team: Authoring Team: Platform
Milestone

Comments

@boydc2014
Copy link
Contributor

boydc2014 commented Oct 15, 2020

  1. Provide new choice in recognizer (orchestrator) with the designed tooltip and help link.

  2. Create a new filed in settings page for the base model selection (by default is the least layer bert-3layers)

  3. Download the base model to a shared location. Consider both web scenario and electron scenario.

  4. Generate the recognizer dialogs once user choose Orchestrator, related PR: Further revision to form element order #435

  5. Do the orchestrator:build + luis:build based on the dialog choices (@lei9444)

  6. Verify it works locally (at this time, remote deploy should not work, because base model missing in remote)

@boydc2014 boydc2014 added Needs-triage A new issue that require triage Area: Orchestrator labels Oct 15, 2020
@boydc2014 boydc2014 added Team: Platform Team: Authoring and removed Needs-triage A new issue that require triage labels Oct 15, 2020
@cwhitten cwhitten added this to the R11 milestone Oct 20, 2020
@cwhitten cwhitten added the P1 Painful if we don't fix, won't block releasing label Oct 20, 2020
@luhan2017
Copy link
Contributor

Will this impact the qna scenario? Based on my understanding, if use selects Orchestrator, it means he/she selects Ochestrator+QnA, and a CrossTrainedRecognizer will be generated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Orchestrator P1 Painful if we don't fix, won't block releasing Team: Authoring Team: Platform
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants