Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

feat: segment and shot label annotations support #281

Merged
merged 3 commits into from
Sep 3, 2019

Conversation

yoshi-automation
Copy link
Contributor

This PR was generated using Autosynth. 🌈

Here's the log from Synthtool:

synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:a684d40ba9a4e15946f5f2ca6b4bd9fe301192f522e9de4fff622118775f309b
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1beta1.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1beta2.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1beta2.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1p1beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1p1beta1.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1p2beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1p2beta1.
synthtool > Running generator for google/cloud/videointelligence/artman_videointelligence_v1p3beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/video-intelligence-v1p3beta1.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1.js.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1p1beta1.js.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1beta2.js.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1p2beta1.js.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1beta1.js.
synthtool > Replaced 'function mockSimpleGrpcMethod.*\n(.*\n)*?}\n' in test/gapic-v1p3beta1.js.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md

> [email protected] postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node scripts/postinstall || echo "ignore"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon: 
> https://opencollective.com/core-js 
> https://www.patreon.com/zloirock 

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)


> [email protected] postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
added 602 packages from 858 contributors and audited 1616 packages in 14.85s
found 0 vulnerabilities


> @google-cloud/[email protected] fix /tmpfs/src/git/autosynth/working_repo
> eslint --fix '**/*.js'


/tmpfs/src/git/autosynth/working_repo/samples/analyze-streaming-annotation-to-storage.js
  28:15  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze-streaming-labels.js
  26:15  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze-streaming-object.js
  26:15  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze-streaming-safe-search.js
  26:15  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze-streaming-shot-change.js
  26:15  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze.js
   21:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
   78:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  143:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  206:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  263:37  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  308:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  362:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  415:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  485:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/analyze.v1p2beta1.js
   20:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
   89:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  160:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require
  235:25  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/quickstart.js
  20:37  error  "@google-cloud/video-intelligence" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze-streaming-annotation-to-storage.test.js
  19:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze-streaming-labels.test.js
  19:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze-streaming-object.test.js
  19:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze-streaming-safe-search.test.js
  19:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze-streaming-shot-change.test.js
  19:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze.test.js
  20:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/analyze.v1p2beta1.test.js
  20:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/system-test/quickstart.test.js
  19:26  error  "chai" is not found  node/no-missing-require

✖ 27 problems (27 errors, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @google-cloud/[email protected] fix: `eslint --fix '**/*.js'`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the @google-cloud/[email protected] fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2019-08-07T11_29_13_513Z-debug.log
synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 7, 2019
@bcoe bcoe added the status: blocked Resolving the issue is dependent on other work. label Aug 20, 2019
@bcoe
Copy link
Contributor

bcoe commented Aug 20, 2019

should we land this? the comments indicate that the fields should not yet be publicly exposed. CC: @lukesneeringer.

@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 21, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 21, 2019
@codecov
Copy link

codecov bot commented Aug 21, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #281   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           9      9           
=====================================
  Hits            9      9

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 0a1363c...b286067. Read the comment docs.

@callmehiphop callmehiphop changed the title [CHANGE ME] Re-generated to pick up changes in the API or client library generator. feat: segment and shot label annotations support Aug 28, 2019
@callmehiphop
Copy link
Contributor

@bcoe any updates here?

@alexander-fenster
Copy link
Contributor

I talked to the team, the feature is going to be released soon but the protos are already public, so I'm guessing it's OK to merge it now.

@alexander-fenster alexander-fenster merged commit 1787c6a into master Sep 3, 2019
@alexander-fenster alexander-fenster deleted the autosynth branch September 3, 2019 21:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement. status: blocked Resolving the issue is dependent on other work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants