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

Added support for interim results #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

elshor
Copy link

@elshor elshor commented Aug 12, 2015

event for interim results is fired when property interimResults of
SpeechRecognition is set to true

event for interim results is fired when property interimResults of
SpeechRecognition is set to true
@elshor elshor closed this Aug 12, 2015
@elshor elshor reopened this Aug 12, 2015
@elshor
Copy link
Author

elshor commented Nov 18, 2015

Sorry but I'm only working with android

@kitkimwong
Copy link

@elshor Thanks you, but one issue:
the result callback:
event.results[0].isFinal ->return undefined
event.resultIndex; ->return undefined
See https://dvcs.w3.org/hg/speech-api/raw-file/tip/webspeechapi.html

So how to check the result is a final result?

Best regards

@elshor
Copy link
Author

elshor commented Nov 19, 2015

I didn't want to diverge from existing (non-compliant) implementation in
which result is implemented as array. Therefore, each alternative has a
boolean field final - results[0][0].final.
It might make sense to update the plugin so it will conform with the spec

On Wed, Nov 18, 2015 at 11:14 PM, kitkimwong [email protected]
wrote:

@elshor https://github.com/elshor Thanks you, but one issue:
the result callback:
event.results[0].isFinal ->return undefined
event.resultIndex; ->return undefined
See https://dvcs.w3.org/hg/speech-api/raw-file/tip/webspeechapi.html


Reply to this email directly or view it on GitHub
#20 (comment)
.

changed implementation so result json will conform with spec so result
will have isFinal field and will not by an array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants