Add streaming microphone sample for Speech #87

CallistoCF · 2018-06-08T21:18:00Z

For @Beccca preliminary review

Command line microphone streaming example in node, run MicrophoneStream.js --help

googlebot · 2018-06-08T21:18:05Z

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.

What to do if you already signed the CLA

Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

codecov · 2018-06-08T21:23:41Z

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #87   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          34     34           
=====================================
  Hits           34     34

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2c42b5...97aec14. Read the comment docs.

…est.

samples/system-test/microphonestream.test.js

+const cwd = path.join(__dirname, `..`);
+const filename = `ManualTest.wav`;
+const filepath = path.join(__dirname, `../resources/${filename}`);
+const text = `neewer manual test 10:39 Friday`;


samples/MicrophoneStream.js

+        .map(result => result.alternatives[0].confidence)
+        .join('\n');
+      console.log(`wavFileToText: Confidence; \n`, confidence);
+      // ResultsArray.push([transcription, confidence]);


samples/system-test/microphonestream.test.js

+
+const {runAsync} = require(`@google-cloud/nodejs-repo-tools`);
+
+//const bucketName = `nodejs-docs-samples-test-${uuid.v4()}`;


samples/system-test/microphonestream.test.js

+'use strict';
+
+const path = require(`path`);
+//const storage = require(`@google-cloud/storage`)();


samples/system-test/microphonestream.test.js

@@ -0,0 +1,45 @@
+/**
+ * Copyright 2016, Google, Inc.


samples/MicStream.js

@@ -0,0 +1,332 @@
+/**


samples/MicStream.js

+  .demand(1)
+  .command(
+    `run`,
+    `Runs complete streaming test: \n 1. streams from microphone, outputting streamed speech-to-text to console, \n 2. records that streamed sample to resources folder, \n 3. runs speech-to-text on that local file to compare to step 1.`,


beccasaurus · 2018-06-13T21:39:41Z

Where are the tests?

samples/MicStream.js

+  setTimeout(function() {
+    record.stop();
+    console.log(`micStreamRecognize: Recording complete!`);
+    console.log(


samples/MicStream.js

+  // [END micStreamRecognize]
+}
+
+function wavFileToText(filename, encoding, sampleRateHertz, languageCode) {


samples/MicStream.js

+  // [END wavFileToText]
+}
+
+function test_1(


samples/MicStream.js

+  var InterimResults = [];
+
+  //defensive code in case user doesn't specify audio file type
+  if (filename.substr(filename.length - 4) !== `.wav`)


samples/MicStream.js

+  // Creates a client
+  const client = new speech.SpeechClient();
+
+  //Creates an array of interim results, for testing


samples/MicStream.js

+
+  //Node-Record-lpcm16
+  const record = require('node-record-lpcm16');
+  //https://www.npmjs.com/package/node-record-lpcm16


samples/MicStream.js

+    interimResults: true, // If you want interim results, set this to true
+  };
+
+  //To write .wav to file


samples/MicStream.js

+
+  const request = {
+    config,
+    interimResults: true, // If you want interim results, set this to true


samples/MicStream.js

+  languageCode
+) {
+  // [START micStreamRecognize]
+  //returns speech-to-text, from streamingRecognize - request object is simplified


samples/MicStream.js

+  // [START micStreamRecognize]
+  //returns speech-to-text, from streamingRecognize - request object is simplified
+  //streams audio from microphone to filename, for secondsofRecording seconds, using simplified request object
+  //config is .wav by default, change request object (see above) for more options.


samples/MicStream.js

+) {
+  // [START micStreamRecognize]
+  //returns speech-to-text, from streamingRecognize - request object is simplified
+  //streams audio from microphone to filename, for secondsofRecording seconds, using simplified request object


samples/MicStream.js

+ * More Info: https://cloud.google.com/speech-to-text/docs/streaming-recognize
+ */
+
+//uncomment below and see Errata at end of file for more configurations


happyhuman

LGTM after fixing my two minor comments.

samples/MicrophoneStream.js

@@ -0,0 +1,129 @@
+/**
+ * Copyright 2017, Google, Inc.


samples/system-test/MicrophoneStream.test.js

@@ -0,0 +1,30 @@
+/**
+ * Copyright 2016, Google, Inc.


JustinBeckwith · 2018-09-10T15:55:05Z

@Beccca @happyhuman 👋 What's the status of this PR? It's been hanging around for a while :) Anything we can do to help get it merged?

googlebot · 2018-09-10T16:07:34Z

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

README.md

@@ -2,149 +2,140 @@
 [//]: # "To regenerate it, use `npm run generate-scaffolding`."
 <img src="https://avatars2.githubusercontent.com/u/2810941?v=3&s=96" alt="Google Cloud Platform logo" title="Google Cloud Platform" align="right" height="96" width="96"/>

-# [Google Cloud Speech API: Node.js Client](https://github.com/googleapis/nodejs-speech)
+# Google Cloud Speech API: Node.js Samples


MicrophoneStreamTest, and basic .ava tests-

b823117

googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jun 8, 2018

Just cosmetic, removed some notes, added note about Event style for t…

53cd4f0

…est.

beccasaurus self-assigned this Jun 11, 2018

beccasaurus reviewed Jun 11, 2018

View reviewed changes

samples/system-test/microphonestream.test.js Outdated

const cwd = path.join(__dirname, `..`);

const filename = `ManualTest.wav`;

const filepath = path.join(__dirname, `../resources/${filename}`);

const text = `neewer manual test 10:39 Friday`;

This comment was marked as spam.

Sign in to view

beccasaurus reviewed Jun 11, 2018

View reviewed changes

samples/MicrophoneStream.js Outdated

.map(result => result.alternatives[0].confidence)

.join('\n');

console.log(`wavFileToText: Confidence; \n`, confidence);

// ResultsArray.push([transcription, confidence]);

This comment was marked as spam.

Sign in to view

beccasaurus reviewed Jun 12, 2018

View reviewed changes

samples/system-test/microphonestream.test.js Outdated

const {runAsync} = require(`@google-cloud/nodejs-repo-tools`);

//const bucketName = `nodejs-docs-samples-test-${uuid.v4()}`;

This comment was marked as spam.

Sign in to view

beccasaurus reviewed Jun 12, 2018

View reviewed changes

samples/system-test/microphonestream.test.js Outdated

'use strict';

const path = require(`path`);

//const storage = require(`@google-cloud/storage`)();

This comment was marked as spam.

Sign in to view

beccasaurus reviewed Jun 12, 2018

View reviewed changes

samples/system-test/microphonestream.test.js Outdated

@@ -0,0 +1,45 @@

/**

* Copyright 2016, Google, Inc.

This comment was marked as spam.

Sign in to view

beccasaurus reviewed Jun 12, 2018

View reviewed changes

samples/MicStream.js Outdated

@@ -0,0 +1,332 @@

/**

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

cfrater and others added 3 commits June 12, 2018 16:05

removed .test, cleaned up yaaarghs

d13734f

Cleaned up test within MicStream.js, .ava tests forthcoming

709638b

Merge branch 'master' into master

a27324c

ghost assigned JustinBeckwith Jun 13, 2018

JustinBeckwith removed their assignment Jun 13, 2018