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

[speech-to-text] calling multiple times to recognize using WebSockets should create individual sessions #187

Closed
germanattanasio opened this issue Dec 21, 2015 · 5 comments
Assignees

Comments

@germanattanasio
Copy link
Contributor

Looking at https://developer.ibm.com/answers/questions/244090/multiple-speech-to-text-consecutive-queries.html

I think the problem is that we call v1/recognize multiple times. We should create and use a session when calling recognize. That session will be using WebSockets.

@daniel-bolanos what do you think ?

@germanattanasio
Copy link
Contributor Author

@nfriedly whenever you have a change

@nfriedly
Copy link
Contributor

I don't think we should create a session unless the user explicitly requests one. I left some comments on the original question, but I haven't been able to reproduce the bug using his code and node.js version (v0.10.29 - which is technically no longer supported, but still worked as correctly for me)

@daniel-bolanos
Copy link

@germanattanasio @nfriedly , what do you mean with sessions? If you mean an STT session, please lets stay away from that... On WebSockets you create the connection calling wss://.../recognize and then every time the user can push multiple files, or chunks of audio over that same connection by sending binary messages. Do you need functionality different than that?

thank you

@nfriedly
Copy link
Contributor

Yea, I don't think that we need stt sessions with websockets, and given that German closed the issue already, I think he agrees.

@daniel-bolanos
Copy link

good :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants