Skip to content
This repository was archived by the owner on Dec 4, 2023. It is now read-only.
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
d311b54
Adding first Iteration of Luis Recognizer
munozemilio Feb 3, 2021
140b56c
Merge branch 'main' into emimunoz/luis-recognizer
munozemilio Feb 3, 2021
293ed8f
Adding tests, comments and fixing bugs
munozemilio Feb 9, 2021
876b517
Enabling live testing, commenting Dialog Recognizer dependency
munozemilio Feb 9, 2021
d7b34ed
Merge branch 'main' into emimunoz/luis-recognizer
munozemilio Feb 9, 2021
2a3413c
Removing spaces and fixing not implemented try catch
munozemilio Feb 9, 2021
583bf50
Fixing linting errors
munozemilio Feb 9, 2021
2b4379f
Fixing style check errors
munozemilio Feb 9, 2021
c22eb94
Fixing style checks
munozemilio Feb 9, 2021
a3d989e
Adding package-info.java file
munozemilio Feb 9, 2021
e67049e
Adding missing Doc on Constructor
munozemilio Feb 9, 2021
e44a01c
Adding missing javadoc at classes
munozemilio Feb 9, 2021
82f5a33
Adding test on send trace Activity
munozemilio Feb 10, 2021
fe9c610
Adding missing test
munozemilio Feb 10, 2021
d1d640f
Merge branch 'main' into emimunoz/luis-recognizer
munozemilio Feb 10, 2021
3ba1ab1
Merge branch 'main' into emimunoz/luis-recognizer
tracyboehrer Feb 10, 2021
c82f737
Added Dialogs Recognizer
tracyboehrer Feb 10, 2021
83fb8cc
Uncommented ExternalEntityRecognizer accessors
tracyboehrer Feb 10, 2021
3944b78
Added missing Dialog.Recognizer.recognize method
tracyboehrer Feb 10, 2021
3a0551f
Adding test for DialogContext scenarios
munozemilio Feb 10, 2021
3add3af
Merge branch 'emimunoz/luis-recognizer' of https://github.com/microso…
munozemilio Feb 10, 2021
32bcec6
Merge branch 'main' into emimunoz/luis-recognizer
munozemilio Feb 10, 2021
651ff8f
Adding missing javadoc
munozemilio Feb 11, 2021
aecde2a
Throwing Exception
munozemilio Feb 11, 2021
4b13fe2
Fixing style
munozemilio Feb 11, 2021
e18618c
Missing error message
munozemilio Feb 11, 2021
4cf56ad
Merge branch 'main' into emimunoz/luis-recognizer
munozemilio Feb 11, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added missing Dialog.Recognizer.recognize method
  • Loading branch information
tracyboehrer committed Feb 10, 2021
commit 3944b78d681a90da966e97605d0150f83d51e7ad
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@
import com.microsoft.bot.builder.BotTelemetryClient;
import com.microsoft.bot.builder.IntentScore;
import com.microsoft.bot.builder.NullBotTelemetryClient;
import com.microsoft.bot.builder.RecognizerConvert;
import com.microsoft.bot.builder.RecognizerResult;
import com.microsoft.bot.builder.TurnContext;
import com.microsoft.bot.connector.Async;
import com.microsoft.bot.schema.Activity;
import com.microsoft.bot.schema.Serialization;
Expand Down Expand Up @@ -101,6 +103,35 @@ public CompletableFuture<RecognizerResult> recognize(
return Async.completeExceptionally(new NotImplementedException("recognize"));
}

/**
* Runs current DialogContext.TurnContext.Activity through a recognizer and returns a
* strongly-typed recognizer result using RecognizerConvert.
*
* @param dialogContext Dialog Context.
* @param activity activity to recognize.
* @param telemetryProperties The properties to be included as part of the event tracking.
* @param telemetryMetrics The metrics to be included as part of the event tracking.
* @param c Class of type T
* @param <T> The RecognizerConvert
* @return
*/
public <T extends RecognizerConvert> CompletableFuture<T> recognize(
DialogContext dialogContext,
Activity activity,
Map<String, String> telemetryProperties,
Map<String, Double> telemetryMetrics,
Class<T> c
) {
return Async.tryCompletable(() -> {
T result = c.newInstance();
return recognize(dialogContext, activity, telemetryProperties, telemetryMetrics)
.thenApply(recognizerResult -> {
result.convert(recognizerResult);
return result;
});
});
}

/**
* Returns ChooseIntent between multiple recognizer results.
*
Expand Down