-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Cloud Speech veneer should not throw an exception if there is more than one result. #2947
Comments
Thanks @pbogle! For I was able to confirm tests with multiple results with the other methods. Do you have an example I could try? |
Thank you for your help. This will be a beta blocker. In general, please check with a cloud speech team member before making any assumptions that aren't explicitly documented in the API. (Feel free to contact me, I'm tech lead for the team.) Even if we didn't return multiple results today, we might start doing so tomorrow if allowed by the specification. An example of a file that results in multiple results is this one Could you help me identify other places where the veener simplifies or flattens results in ways that aren't explicitly guaranteed to by the spec? We should look at those as potential sources of bugs. |
Awesome! Thanks! I'll see if I can find other places that flatten out the data. There may be one or two. Do you happen to know who your Vision API counterpart would be? |
@daspecster I am unable to change assignment on this issue, but am currently handling it. |
Thanks @lukesneeringer! I cleared the assignees but I can't assign you. Must be a permissions thing for your account? |
PR is #2962, waiting on CI and approval. |
This is actually pretty broken now. The system tests exhibit the issues. |
This has since been dealt with by other PRs; re-closing. :-) |
I believe this code in the veneer is incorrect because it throws if there is more than one result.
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/speech/google/cloud/speech/_gax.py#L263
As documented in the proto, the results contain a sequential list of transcription results corresponding to sequential portions of audio.
The text was updated successfully, but these errors were encountered: