Merge pull request #122 from spokestack/jz-kw-timeout

fix: timeout event when no keyword is recognized
spokestack · Dec 11, 2020 · 771d085 · 771d085
2 parents 9618e59 + 326b427
commit 771d085
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/src/main/java/io/spokestack/spokestack/asr/KeywordRecognizer.java b/src/main/java/io/spokestack/spokestack/asr/KeywordRecognizer.java
@@ -429,17 +429,16 @@ private void detect(SpeechContext context) {
 
         context.traceInfo("keyword: %.3f %s", confidence, transcript);
 
-        // if we are under threshold, return the null class
-        if (confidence < this.threshold) {
-            transcript = "";
-            confidence = 1.0f - confidence;
+        if (confidence >= this.threshold) {
+            // raise the speech recognition event with the class transcript
+            context.setTranscript(transcript);
+            context.setConfidence(confidence);
+            context.dispatch(SpeechContext.Event.RECOGNIZE);
+        } else {
+            // if we are under threshold, time out without recognition
+            context.dispatch(SpeechContext.Event.TIMEOUT);
         }
 
-        // raise the speech recognition event with the class transcript
-        context.setTranscript(transcript);
-        context.setConfidence(confidence);
-        context.dispatch(SpeechContext.Event.RECOGNIZE);
-
         reset();
     }
 

diff --git a/src/test/java/io/spokestack/spokestack/asr/KeywordRecognizerTest.java b/src/test/java/io/spokestack/spokestack/asr/KeywordRecognizerTest.java
@@ -111,7 +111,7 @@ public void testRecSilentCtxActive() throws Exception {
 
     @Test
     public void testRecNullCtxDeactivate() throws Exception {
-        // verify that the null transcript is raised for the null class
+        // verify that a timeout is raised for the null class
         TestEnv env = new TestEnv(testConfig());
 
         env.context.setActive(true);
@@ -120,7 +120,7 @@ public void testRecNullCtxDeactivate() throws Exception {
         env.context.setActive(false);
         env.process();
 
-        assertEquals(SpeechContext.Event.RECOGNIZE, env.event);
+        assertEquals(SpeechContext.Event.TIMEOUT, env.event);
         assertEquals("", env.context.getTranscript());
     }