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

Add sample for speech api on GCS file via grpc. #352

Merged
merged 1 commit into from
May 18, 2016
Merged

Conversation

jerjou
Copy link
Contributor

@jerjou jerjou commented May 17, 2016

Note that grpc doesn't yet work on python3, so the test is skipped.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 17, 2016
import argparse

from gcloud.credentials import get_credentials
from google.cloud.speech.v1.cloud_speech_pb2 import * # noqa
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you done this elsewhere? please never do this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah - I don't usually, but the auto-generated protobuf libs are pretty opaque about what exactly they declared -_-;

Actually, I realize that I can just import the module. I'll do that... LMKWYT

@theacodes
Copy link
Contributor

Please make travis happy.

@jerjou jerjou force-pushed the speech-gcs branch 2 times, most recently from 45b89ee to 0f3d214 Compare May 18, 2016 18:17

import contextlib
import re
import threading

from gcloud.credentials import get_credentials
from google.cloud.speech.v1.cloud_speech_pb2 import * # noqa
from google.cloud.speech.v1 import cloud_speech_pb2 as pb
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rather you not alias it. If you are going to alias it, use cloud_speech.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@theacodes
Copy link
Contributor

LGTM once travis is happy.

@jerjou jerjou merged commit c5c6c95 into master May 18, 2016
@jerjou jerjou deleted the speech-gcs branch May 18, 2016 18:35
@jerjou
Copy link
Contributor Author

jerjou commented May 18, 2016

Me and travis are tight.

rsamborski pushed a commit that referenced this pull request Nov 8, 2022
rsamborski pushed a commit that referenced this pull request Nov 8, 2022
rsamborski pushed a commit that referenced this pull request Nov 11, 2022
rsamborski pushed a commit that referenced this pull request Nov 14, 2022
dandhlee pushed a commit that referenced this pull request Nov 14, 2022
telpirion pushed a commit that referenced this pull request Nov 16, 2022
m-strzelczyk pushed a commit that referenced this pull request Nov 18, 2022
dandhlee pushed a commit that referenced this pull request Nov 18, 2022
engelke pushed a commit that referenced this pull request Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants