Skip to content

Gh279 streaming example #287

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

Merged
merged 30 commits into from
Nov 13, 2017
Merged

Gh279 streaming example #287

merged 30 commits into from
Nov 13, 2017

Conversation

mediumTaj
Copy link
Contributor

Summary

Update Speech to Text streaming example

@kimberlysiva
Copy link
Contributor

@mediumTaj Just noticed that the maxLevel logic was reverted again. Both streaming samples need to be updated to use Mathf.Max(Mathf.Abs(Mathf.Min(samples)), Mathf.Max(samples))

@kimberlysiva
Copy link
Contributor

@mediumTaj Also, we lost the change to _sendEvent.WaitOne(50); instead of 500.

@mediumTaj
Copy link
Contributor Author

good catches, thanks!

@mediumTaj
Copy link
Contributor Author

@kimberlysiva I'm planning to merge this PR in today - if you have some time can you review it and let me know if we are missing anything?

@kimberlysiva
Copy link
Contributor

@mediumTaj I've been testing it today and things look good (also on Hololens, very responsive!)

I think we may want to rename ExampleStreamingChunked, since at first glance this suggests a REST connection with chunked encoding.

@mediumTaj
Copy link
Contributor Author

Awesome thanks! Chunked rename is done and pushed. After we pass CI, I'll merge this in.

@mediumTaj mediumTaj merged commit 4a30f92 into develop Nov 13, 2017
@mediumTaj mediumTaj deleted the gh279-streaming-example branch November 13, 2017 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants