Skip to content
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.

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

migrate code from googleapis/python-videointelligence #10605

Merged
merged 233 commits into from
Sep 15, 2023

Conversation

dizcology
Copy link
Member

migrate code from python-videointelligence

Description

Fixes #

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

gguuss and others added 30 commits September 30, 2020 13:21
* Adds tutorials.

* Removes unused enumerate
…945)

* Adds new examples, replaces markdown with restructured text

* Address review feedback

* Use videos from pubilc bucket, update to new client library.

* Style nit
* Fix README rst links

* Update all READMEs
updated the gcs files used in Usage to use public ones. Hopefully the readme will be updated
* Auto-update dependencies.

* Fix storage notification poll sample

Change-Id: I6afbc79d15e050531555e4c8e51066996717a0f3

* Fix spanner samples

Change-Id: I40069222c60d57e8f3d3878167591af9130895cb

* Drop coverage because it's not useful

Change-Id: Iae399a7083d7866c3c7b9162d0de244fbff8b522

* Try again to fix flaky logging test

Change-Id: I6225c074701970c17c426677ef1935bb6d7e36b4
Change-Id: I03b5eaef8b17ac3dc3c0339fd2c7447bd3e11bd2
* update analyze_safe_search

* update analyze_shots

* update explicit_content_detection and test

* update fece detection

* update label detection (path)

* update label detection (file)

* flake

* safe search --> explicit content

* update faces tutorial

* update client library quickstart

* update shotchange tutorial

* update labels tutorial

* correct spelling

* correction start_time_offset

* import order

* rebased
* Update Readme.rst to add Python setup guide

As requested in b/64770713.

This sample is linked in documentation https://cloud.google.com/bigtable/docs/scaling, and it would make more sense to update the guide here than in the documentation.

* Update README.rst

* Update README.rst

* Update README.rst

* Update README.rst

* Update README.rst

* Update install_deps.tmpl.rst

* Updated readmegen scripts and re-generated related README files

* Fixed the lint error
* update samples to v1

* replace while loop with operation.result(timeout)

* addressing review comments

* flake

* flake
Protobuf handles the base64 encoding for you, and so the content was being base64 encoded twice.
* Auto-update dependencies.

* Update requirements.txt
* remove face detection samples

* update docstring

* linter

* linter
* Get display name of enums using IntEnum

Requires updating google-cloud-language to 1.1.0

* Add note about gs://demomaker for video test files

* Get display name of enums using IntEnum

* Get display name of enums using IntEnum

* Revert "Add note about gs://demomaker for video test files"

This reverts commit 39d9bff.
renovate-bot and others added 9 commits April 18, 2023 12:00
* fix: expand region tags to include `import io`
Source-Link: https://github.com/googleapis/synthtool/commit/30bd01b4ab78bf1b2a425816e15b3e7e090993dd
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:9bc5fa3b62b091f60614c08a7fb4fd1d3e1678e326f34dd66ce1eefb5dc3267b
* chore(deps): update dependency google-cloud-storage to v2.9.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…1.2 (#425)

Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…1.3 (#435)

Co-authored-by: Anthonios Partheniou <partheniou@google.com>
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Sep 11, 2023
@google-cla
Copy link

google-cla bot commented Sep 11, 2023

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

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@dizcology dizcology marked this pull request as ready for review September 11, 2023 22:15
@dizcology dizcology requested review from a team as code owners September 11, 2023 22:15
@snippet-bot
Copy link

snippet-bot bot commented Sep 11, 2023

Here is the summary of changes.

You are about to add 40 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

Copy link
Member

@rsamborski rsamborski left a comment

Choose a reason for hiding this comment

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

Thank you for the code migration. Before merging please update:

Couple additional notes to remember:

  • CLA issues will require admin merge
  • Remember to use merge instead of squash and merge to keep history intact

videointelligence/samples/analyze/analyze_test.py Outdated Show resolved Hide resolved
@dizcology
Copy link
Member Author

CODEOWNERS and blunderbuss updated.

@leahecole
Copy link
Collaborator

CLA is okay, these are xooglers

@leahecole leahecole merged commit fd0dd89 into main Sep 15, 2023
4 checks passed
@leahecole leahecole deleted the python-videointelligence-migration branch September 15, 2023 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.