Skip to content

Diarization - #1556

Merged
happyhuman merged 15 commits into
masterfrom
diarization
Jul 12, 2018
Merged

Diarization#1556
happyhuman merged 15 commits into
masterfrom
diarization

Conversation

@happyhuman

Copy link
Copy Markdown
Contributor

The samples in this code demo diarization: Basically who said what (when there is more than just one person talking).

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 10, 2018
@happyhumanhappyhuman changed the title Diarization[DO NOT MERGE] DiarizationJul 10, 2018
@happyhuman

Copy link
Copy Markdown
ContributorAuthor

@nnegrey , @dizcology , @theacodes Can you please review this PR?

@nnegrey

Copy link
Copy Markdown
Contributor

Looks like these use the speech_v1p1beta1 library, can we put these snippets into the beta_snippets.py file?

python transcribe_diarization.py \
resources/Google_Gnome.wav
python transcribe_diarization.py \
gs://cloud-ml-api-e2e-testing/speech/stereo_audio.wav

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cloud-ml-api-e2e-testing public bucket?

@happyhuman

Copy link
Copy Markdown
ContributorAuthor

Good catch @nnegrey. I moved all the 4 beta samples to beta_snippets.py and simplified a few things.

@happyhuman
happyhuman requested a review from tswastJuly 12, 2018 21:10
@happyhumanhappyhuman changed the title [DO NOT MERGE] DiarizationDiarizationJul 12, 2018
@happyhuman
happyhuman merged commit ad4ae03 into masterJul 12, 2018
print('First alternative of result {}: {}'
.format(i, alternative.transcript))
print('Speaker Tag for the first word: {}'
.format(alternative.words[0].speaker_tag))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does each word in alternative.words have other relevant information concerning diarization besides speaker_tag? if so print them as well, if not please ignore this comment.

os.path.join(RESOURCES, 'Google_Gnome.wav'))
out, err = capsys.readouterr()

assert 'OK Google stream stranger things from Netflix to my TV' in out

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

are there two different speakers in this audio file? if so assert something about the speaker_tags being returned correctly.

@happyhuman
happyhuman deleted the diarization branch July 13, 2018 23:17
busunkim96 pushed a commit to busunkim96/python-speech that referenced this pull request Sep 1, 2020
busunkim96 pushed a commit to googleapis/python-speech that referenced this pull request Sep 3, 2020
telpirion pushed a commit that referenced this pull request Jan 13, 2023
dandhlee pushed a commit that referenced this pull request Feb 9, 2023
telpirion pushed a commit that referenced this pull request Mar 13, 2023
atulep pushed a commit to googleapis/google-cloud-python that referenced this pull request Apr 6, 2023
atulep pushed a commit to googleapis/google-cloud-python that referenced this pull request Apr 6, 2023
atulep pushed a commit to googleapis/google-cloud-python that referenced this pull request Apr 18, 2023
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Oct 22, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@happyhuman@nnegrey@dizcology@googlebot