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.

Sign up for GitHub

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

Jump to bottom

Add async support to Speech API #2384

Closed
daspecster opened this issue Sep 22, 2016 · 5 comments
Closed

Add async support to Speech API #2384

daspecster opened this issue Sep 22, 2016 · 5 comments
Assignees
@daspecster
Labels
api: speech Issues related to the Speech-to-Text API.

Comments

@daspecster
Copy link
Contributor

Add async support to Google Cloud Speech API.
@daspecster daspecster added the api: speech Issues related to the Speech-to-Text API. label Sep 22, 2016
@daspecster daspecster self-assigned this Sep 22, 2016
@daspecster
Copy link
Contributor Author

@dhermes @tseaver

The Operation response for Speech async requests seems pretty generic and possibly similar to google.cloud.operation.Operation.

Should I just add a non-protobuf response parser toOperation? (instead of from_pb() a from_api_repr).

@daspecster
Copy link
Contributor Author

The more I look at it, they seem very different. I'm going to just make speech.operation unless you think I should put it in google.cloud.operation.

@dhermes
Copy link
Contributor

dhermes commented Sep 22, 2016

Check out the proto for the service. It's the very same Operation used by Bigtable and others.

@daspecster
Copy link
Contributor Author

daspecster commented Sep 22, 2016 •
edited
Loading

Speech sync recognize is implemented via JSON/REST at the moment.

Should I just implement asyncrecognize with gapic GAX? This seems weird to me since, grpc is an optional requirement. (Was my usage of gapic and grpc correct in that wording?)

Also it appears that gapic isn't available for speech yet. GAX and GRPC are though.

@daspecster
Copy link
Contributor Author

Done with #2663(GAX) and #2426(JSON/REST).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@daspecster daspecster

Labels
api: speech Issues related to the Speech-to-Text API.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@daspecster @dhermes