Video: faster videos + better print statements + use NPM module #395

ace-n · 2017-05-18T04:49:01Z

No description provided.

gguuss · 2017-05-18T04:52:48Z

video/quickstart.js

  projectId: process.env.GCLOUD_PROJECT // Replace with your Google Cloud project ID
 });

 // The GCS filepath of the video to analyze
-const gcsUri = 'gs://demomaker/volleyball_court.mp4';
+const gcsUri = 'gs://demomaker/tomatoes.mp4';


No action required, but I'm using the ML videos from the video landing page, examples here.

Noted - is this just a case of s/demomaker/cloudmleap/video/next?

codecov-io · 2017-05-18T04:55:13Z

Codecov Report

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

@@           Coverage Diff           @@
##           master     #395   +/-   ##
=======================================
  Coverage   83.88%   83.88%           
=======================================
  Files           4        4           
  Lines         422      422           
=======================================
  Hits          354      354           
  Misses         68       68

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 8b6b0a2...d4b0e99. Read the comment docs.

gguuss

LGTM. I'm taking a todo to come up with a good video for us to publicly host for testing.

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [uuid](https://togithub.com/uuidjs/uuid) | devDependencies | major | [`^7.0.0` -> `^8.0.0`](https://renovatebot.com/diffs/npm/uuid/7.0.3/8.0.0) | --- ### Release Notes <details> <summary>uuidjs/uuid</summary> ### [`v8.0.0`](https://togithub.com/uuidjs/uuid/blob/master/CHANGELOG.md#800-httpsgithubcomuuidjsuuidcomparev703v800-2020-04-29) [Compare Source](https://togithub.com/uuidjs/uuid/compare/v7.0.3...v8.0.0) ##### ⚠ BREAKING CHANGES - For native ECMAScript Module (ESM) usage in Node.js only named exports are exposed, there is no more default export. ```diff -import uuid from 'uuid'; -console.log(uuid.v4()); // -> 'cd6c3b08-0adc-4f4b-a6ef-36087a1c9869' +import { v4 as uuidv4 } from 'uuid'; +uuidv4(); // ⇨ '9b1deb4d-3b7d-4bad-9bdd-2b0d7b3dcb6d' ``` - Deep requiring specific algorithms of this library like `require('uuid/v4')`, which has been deprecated in `uuid@7`, is no longer supported. Instead use the named exports that this module exports. For ECMAScript Modules (ESM): ```diff -import uuidv4 from 'uuid/v4'; +import { v4 as uuidv4 } from 'uuid'; uuidv4(); ``` For CommonJS: ```diff -const uuidv4 = require('uuid/v4'); +const { v4: uuidv4 } = require('uuid'); uuidv4(); ``` ##### Features - native Node.js ES Modules (wrapper approach) ([#423](https://togithub.com/uuidjs/uuid/issues/423)) ([2d9f590](https://togithub.com/uuidjs/uuid/commit/2d9f590ad9701d692625c07ed62f0a0f91227991)), closes [#245](https://togithub.com/uuidjs/uuid/issues/245) [#419](https://togithub.com/uuidjs/uuid/issues/419) [#342](https://togithub.com/uuidjs/uuid/issues/342) - remove deep requires ([#426](https://togithub.com/uuidjs/uuid/issues/426)) ([daf72b8](https://togithub.com/uuidjs/uuid/commit/daf72b84ceb20272a81bb5fbddb05dd95922cbba)) ##### Bug Fixes - add CommonJS syntax example to README quickstart section ([#417](https://togithub.com/uuidjs/uuid/issues/417)) ([e0ec840](https://togithub.com/uuidjs/uuid/commit/e0ec8402c7ad44b7ef0453036c612f5db513fda0)) ##### [7.0.3](https://togithub.com/uuidjs/uuid/compare/v7.0.2...v7.0.3) (2020-03-31) ##### Bug Fixes - make deep require deprecation warning work in browsers ([#409](https://togithub.com/uuidjs/uuid/issues/409)) ([4b71107](https://togithub.com/uuidjs/uuid/commit/4b71107d8c0d2ef56861ede6403fc9dc35a1e6bf)), closes [#408](https://togithub.com/uuidjs/uuid/issues/408) ##### [7.0.2](https://togithub.com/uuidjs/uuid/compare/v7.0.1...v7.0.2) (2020-03-04) ##### Bug Fixes - make access to msCrypto consistent ([#393](https://togithub.com/uuidjs/uuid/issues/393)) ([8bf2a20](https://togithub.com/uuidjs/uuid/commit/8bf2a20f3565df743da7215eebdbada9d2df118c)) - simplify link in deprecation warning ([#391](https://togithub.com/uuidjs/uuid/issues/391)) ([bb2c8e4](https://togithub.com/uuidjs/uuid/commit/bb2c8e4e9f4c5f9c1eaaf3ea59710c633cd90cb7)) - update links to match content in readme ([#386](https://togithub.com/uuidjs/uuid/issues/386)) ([44f2f86](https://togithub.com/uuidjs/uuid/commit/44f2f86e9d2bbf14ee5f0f00f72a3db1292666d4)) ##### [7.0.1](https://togithub.com/uuidjs/uuid/compare/v7.0.0...v7.0.1) (2020-02-25) ##### Bug Fixes - clean up esm builds for node and browser ([#383](https://togithub.com/uuidjs/uuid/issues/383)) ([59e6a49](https://togithub.com/uuidjs/uuid/commit/59e6a49e7ce7b3e8fb0f3ee52b9daae72af467dc)) - provide browser versions independent from module system ([#380](https://togithub.com/uuidjs/uuid/issues/380)) ([4344a22](https://togithub.com/uuidjs/uuid/commit/4344a22e7aed33be8627eeaaf05360f256a21753)), closes [#378](https://togithub.com/uuidjs/uuid/issues/378) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/nodejs-automl).

Switch to faster videos + update print statements

d4b0e99

ace-n requested a review from gguuss May 18, 2017 04:49

gguuss reviewed May 18, 2017

View reviewed changes

gguuss approved these changes May 18, 2017

View reviewed changes

ace-n merged commit 2ac3dd6 into master May 18, 2017

ace-n deleted the video-npm branch May 18, 2017 04:59

This was referenced Jul 22, 2021

[Snyk] Security upgrade @google-cloud/debug-agent from 2.1.3 to 2.4.0 ninadhatkar/nodejs-docs-samples#102

Open

[Snyk] Security upgrade @google-cloud/debug-agent from 2.1.3 to 2.4.0 ninadhatkar/nodejs-docs-samples#111

Open

ace-n mentioned this pull request Nov 11, 2022

migrate code from googleapis/nodejs-automl #2841

Merged

ace-n pushed a commit that referenced this pull request Nov 16, 2022

Switch to faster videos + update print statements (#395)

89a7a80

ace-n mentioned this pull request Nov 16, 2022

migrate code from googleapis/nodejs-video-intelligence #2855

Merged

ace-n pushed a commit that referenced this pull request Nov 17, 2022

Switch to faster videos + update print statements (#395)

57d75a9

ahrarmonsur pushed a commit that referenced this pull request Nov 17, 2022

Switch to faster videos + update print statements (#395)

e12d354

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Video: faster videos + better print statements + use NPM module #395

Video: faster videos + better print statements + use NPM module #395

ace-n commented May 18, 2017

gguuss May 18, 2017 •

edited

Loading

ace-n May 18, 2017

codecov-io commented May 18, 2017

gguuss left a comment

Video: faster videos + better print statements + use NPM module #395

Video: faster videos + better print statements + use NPM module #395

Conversation

ace-n commented May 18, 2017

gguuss May 18, 2017 • edited Loading

Choose a reason for hiding this comment

ace-n May 18, 2017

Choose a reason for hiding this comment

codecov-io commented May 18, 2017

Codecov Report

gguuss left a comment

Choose a reason for hiding this comment

gguuss May 18, 2017 •

edited

Loading