scratchfoundation/scratch-vm
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-vm.git synced 2024-12-24 15:02:52 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Merge pull request #1199 from picklesrus/speech-extension-loginfo

Browse source 
Add some log info with more information about the speech match so we …
This commit is contained in:
Eric Rosenbaum 2018-06-04 16:35:58 -04:00 committed by GitHub
commit ca6a7f9311
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 18 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/extensions/scratch3_speech/index.js
View file

@ -81,7 +81,7 @@ class Scratch3SpeechBlocks {
/** /**
* Similar to _currentUtterance, but set back to '' at the beginning of listening block. * Similar to _currentUtterance, but set back to '' at the beginning of listening block.
* Used to get the hat blocks to edge trigger. In order to detect someone saying * Used to get the hat blocks to edge trigger. In order to detect someone saying
* the same thing twice in two subsequent liten and wait blocks * the same thing twice in two subsequent listen and wait blocks
* and still trigger the hat, we need this to go from * and still trigger the hat, we need this to go from
* '' at the beginning of the listen block to '<transcription value>' at the end. * '' at the beginning of the listen block to '<transcription value>' at the end.
* @type {string} * @type {string}
@ -373,9 +373,26 @@ class Scratch3SpeechBlocks {
// If the result from the speech api isn't very stable and we only had a fuzzy match, we don't want to use it. // If the result from the speech api isn't very stable and we only had a fuzzy match, we don't want to use it.
const shouldKeepFuzzyMatch = fuzzyMatchIndex !== -1 && result.stability > stabilityThreshold; const shouldKeepFuzzyMatch = fuzzyMatchIndex !== -1 && result.stability > stabilityThreshold;
// TODO: This is for debugging. Remove when this function is finalized.
if (shouldKeepFuzzyMatch) {
log.info(`Fuzzy match with high stability.`);
log.info(`match index is ${fuzzyMatchIndex}`);
const phrases = this._phraseList.join(' ');
const matchPhrase = phrases.substring(fuzzyMatchIndex, fuzzyMatchIndex + normalizedTranscript.length);
log.info(`fuzzy match: ${matchPhrase} in ${normalizedTranscript}`);
}
// If the result is in the phraseList (i.e. it matches one of the 'When I Hear' blocks), we keep it. // If the result is in the phraseList (i.e. it matches one of the 'When I Hear' blocks), we keep it.
// This might be aggressive... but so far seems to be a good thing. // This might be aggressive... but so far seems to be a good thing.
const shouldKeepPhraseListMatch = this._phraseList.includes(normalizedTranscript); const shouldKeepPhraseListMatch = this._phraseList.includes(normalizedTranscript);
// TODO: This is just for debugging. Remove when this function is finalized.
if (shouldKeepPhraseListMatch) {
log.info(`phrase list ${this._phraseList} includes ${normalizedTranscript}`);
}
// TODO: This is for debugging. Remove when this function is finalized.
if (result.isFinal) {
log.info(`result is final`);
}
if (!result.isFinal && !shouldKeepPhraseListMatch && !shouldKeepFuzzyMatch) { if (!result.isFinal && !shouldKeepPhraseListMatch && !shouldKeepFuzzyMatch) {
return false; return false;