Skip to content

Added setEnableWordTimeOffsets(true) to Async Recognize for a File #781

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

Closed
dlaqab wants to merge 5 commits into from
Closed

Added setEnableWordTimeOffsets(true) to Async Recognize for a File #781

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1e4cf44
Added setEnableWordTimeOffsets(true) to Async Recognize for a File
Aug 1, 2017
a893740
Added missing test for AsyncRecognizeFile
Aug 2, 2017
29f9f76
Merge remote-tracking branch 'upstream/master'
Aug 2, 2017
e4a7cce
Changed Nanoseconds to Seconds, and fixed the tests
Aug 2, 2017
3384179
Merge branch 'master' into dalaqab_enableWordTimeOffsets_examples
gguuss Aug 3, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions speech/cloud-client/src/main/java/com/example/speech/QuickstartSample.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ public static void main(String... args) throws Exception {
.setEncoding(AudioEncoding.LINEAR16)
.setSampleRateHertz(16000)
.setLanguageCode("en-US")
.setEnableWordTimeOffsets(false)

@gguuss gguuss Aug 1, 2017

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.

Isn't this superfluous as false is the default?

@gguuss gguuss Aug 2, 2017

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.

I'm pretty sure we don't want to set an optional parameter to its default value in the quickstart example. Thoughts?

.build();
RecognitionAudio audio = RecognitionAudio.newBuilder()
.setContent(audioBytes)
Expand Down
8 changes: 8 additions & 0 deletions speech/cloud-client/src/main/java/com/example/speech/Recognize.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ public static void syncRecognizeFile(String fileName) throws Exception, IOExcept
.setEncoding(AudioEncoding.LINEAR16)
.setLanguageCode("en-US")
.setSampleRateHertz(16000)
.setEnableWordTimeOffsets(false)

@gguuss gguuss Aug 1, 2017

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.

Isn't false the default?

.build();
RecognitionAudio audio = RecognitionAudio.newBuilder()
.setContent(audioBytes)
Expand Down Expand Up @@ -127,6 +128,7 @@ public static void syncRecognizeGcs(String gcsUri) throws Exception, IOException
.setEncoding(AudioEncoding.FLAC)
.setLanguageCode("en-US")
.setSampleRateHertz(16000)
.setEnableWordTimeOffsets(false)

@gguuss gguuss Aug 1, 2017

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.

Isn't false the default?

.build();
RecognitionAudio audio = RecognitionAudio.newBuilder()
.setUri(gcsUri)
Expand Down Expand Up @@ -165,6 +167,7 @@ public static void asyncRecognizeFile(String fileName) throws Exception, IOExcep
.setEncoding(AudioEncoding.LINEAR16)
.setLanguageCode("en-US")
.setSampleRateHertz(16000)
.setEnableWordTimeOffsets(true)
.build();
RecognitionAudio audio = RecognitionAudio.newBuilder()
.setContent(audioBytes)
Expand All @@ -186,6 +189,11 @@ public static void asyncRecognizeFile(String fileName) throws Exception, IOExcep
List<SpeechRecognitionAlternative> alternatives = result.getAlternativesList();
for (SpeechRecognitionAlternative alternative: alternatives) {
System.out.printf("Transcription: %s%n", alternative.getTranscript());
for (WordInfo wordInfo: alternative.getWordsList()) {

@gguuss gguuss Aug 1, 2017

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.

Please add a test for this.

@dlaqab dlaqab Aug 2, 2017

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done. I added the test. I also fixed the test failure in AsyncRecognizeGcs test, so all the tests should pass now

System.out.println(wordInfo.getWord());
System.out.printf("\t%s ns - %s ns\n",
wordInfo.getStartTime().getNanos(), wordInfo.getEndTime().getNanos());

@gguuss gguuss Aug 1, 2017

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.

You're missing seconds, this just shows the nanoseconds for portions of the response.

@gguuss gguuss Aug 1, 2017

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.

Please update the other async sample to correctly show the seconds and nanos calculated to second fractions.

@dlaqab dlaqab Aug 2, 2017

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

your original code did not have seconds, and I did not change the "pretty print" to show seconds.

}
}
}
speech.close();
Expand Down