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

Make QnA Maker Active Learnings Threasholds user settable #4251

Closed
wants to merge 678 commits into from

Conversation

cleemullins
Copy link
Contributor

The C# version of #3706.
@somigithub has promised to do the JS version tonight.

xieofxie and others added 30 commits May 9, 2020 16:40
* Add generation scripts and interdependency graph

* change 'azure' to 'botbuilder'
* [Expression] Add formatNumber, formatEpoch and  formatTicks as prebuilt functions (#3876) (#3883)

* add formatNumber prebuilt function

* change to string output to keep tailing zeors

* Add formatEpoch and formatTicks.
Removed ticks from formatDate and fixed some bugs.
Fixed a bug in JSON conversion where 64 bit/double values were getting truncated.

* Add tests that were dropped.

* Add locale to formatNumber.

* Add 1000's seperators.

* fix a test case

* change a test case

Co-authored-by: Chris McConnell <[email protected]>

Co-authored-by: Chris McConnell <[email protected]>

* override OnComputeId in AdaptiveSkillDialog (#3882)

* override OnComputeId in AdaptiveSkillDialog

* refactor AdaptiveSkillDialog.OnComputeId()

* remove superfluous parenthesis around activity

* apply @Stevenic's PR feedback

* Rename $kind Microsoft.SkillDialog => Microsoft.BeginSkillDialog
Rename class AdaptiveSkillDialog => BeginSkillDialog
Move files into actions folders
replace all $kind => new $kind

* Style: Async method nomenclature, missing ConfigureAwait and CancellationTokens

* Rename $kind Microsoft.BeginSkillDialog => Microsoft.BeginSkill
Rename class BeginSkillDialog to BeginSkill

* update title for begin skill

* Cancellation, nomenclature and style improvements

* Cancellation tokens and propagation

* merge CheckForVersionChangeAsync()

* Missing cancellation tokens

Co-authored-by: Shuai Wang <[email protected]>
Co-authored-by: Chris McConnell <[email protected]>
Co-authored-by: Tom Laird-McConnell <[email protected]>
Co-authored-by: Carlos Castro <[email protected]>
Co-authored-by: Carlos Castro <[email protected]>
* Fix locale bug in testadapter and conversation reference.
* Add debugComposer class to make it easy to debug a composerbot
…low interruptions.

If QnAMaker is interrupted end the dialog.
Fix QnAMakerDialog to handle interuption scenarios
… non-loops, don't clone on loops - just stitch things together.
DialogSet.Add: Unique dialog ids for name collisions but not for reference collisions
…aced it with nuget.packaing which is many versions past the one we were using.

Worse, nuget.client had a bad prerelase package pushed in 2017 that breaks install adaptive into linqpad.
also fixed .dialog files which were not compliant to schema (missing actions node)
* Do NOT call TeamsInfo.getMember for the bot

* Add TestConversationUpdateBotTeamsMemberAdded
* Add dependency graph

* Update README.md

* Update README.md

* Remove DependencyGraph folder

* Update graph title

* Update graph title 2

* Title tweak 3

* Fix path to graph
On my machine, this unit test fails. The issue is that the expected value is incorrect. The `CurrencyGroupSeparator` for `fr-FR` is `\x202F` not `00a0`
If the server validates that the timestamp is recent to prevent replay attacks, then this would fail.
update resultProperty descption of http request step
@somigithub
Copy link
Contributor

somigithub commented Jul 9, 2020

Thanks for your super quick C# PR.

@cleemullins similar change for JS here: microsoft/botbuilder-js#2508
This is my first bot framework PR :)

@somigithub somigithub closed this Jul 11, 2020
@somigithub somigithub reopened this Jul 11, 2020
@chrimc62 chrimc62 force-pushed the CLM/MakeQnaActiveLearningConstantsSettable branch from 7be43c6 to 48c32bf Compare July 15, 2020 02:04
@cleemullins cleemullins added blocked Current progress is blocked on something else. needs rebase labels Jul 15, 2020
@somigithub
Copy link
Contributor

New PR: #4383

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Current progress is blocked on something else.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make QnA Training Thresholds Configurable