-
Notifications
You must be signed in to change notification settings - Fork 6.5k
video: add auto-generated Logo Recognition Samples #2755
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
Merged
dizcology
merged 9 commits into
GoogleCloudPlatform:master
from
texasmichelle:logo_recognition
Jan 31, 2020
Merged
video: add auto-generated Logo Recognition Samples #2755
dizcology
merged 9 commits into
GoogleCloudPlatform:master
from
texasmichelle:logo_recognition
Jan 31, 2020
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it
…cs-samples into logo_recognition
Each version submits 16 requests, which exceeds standard quota of 60 when running in parallel. I submitted a request for additional quota. |
leahecole
reviewed
Jan 30, 2020
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as @dizcology is okay with it, I'm okay with it. Thank you for figuring out what needed to happen with the quota
dizcology
approved these changes
Jan 31, 2020
busunkim96
pushed a commit
to busunkim96/python-videointelligence
that referenced
this pull request
May 20, 2020
…oudPlatform/python-docs-samples#2755) * Add auto-generated Logo Recognition Samples * Add tests Remove boilerplate Update copyright date Blacken Remove unused imports Shorten docstrings Remove CLI Set defaults in function definition * Edit test video to 3 seconds from 35 Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it * Fix bug Co-authored-by: Noah Negrey <[email protected]> Co-authored-by: Leah E. Cole <[email protected]>
danoscarmike
pushed a commit
to googleapis/python-videointelligence
that referenced
this pull request
Sep 30, 2020
…oudPlatform/python-docs-samples#2755) * Add auto-generated Logo Recognition Samples * Add tests Remove boilerplate Update copyright date Blacken Remove unused imports Shorten docstrings Remove CLI Set defaults in function definition * Edit test video to 3 seconds from 35 Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it * Fix bug Co-authored-by: Noah Negrey <[email protected]> Co-authored-by: Leah E. Cole <[email protected]>
dizcology
pushed a commit
that referenced
this pull request
Sep 11, 2023
* Add auto-generated Logo Recognition Samples * Add tests Remove boilerplate Update copyright date Blacken Remove unused imports Shorten docstrings Remove CLI Set defaults in function definition * Edit test video to 3 seconds from 35 Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it * Fix bug Co-authored-by: Noah Negrey <[email protected]> Co-authored-by: Leah E. Cole <[email protected]>
9 tasks
leahecole
added a commit
that referenced
this pull request
Sep 15, 2023
* Add auto-generated Logo Recognition Samples * Add tests Remove boilerplate Update copyright date Blacken Remove unused imports Shorten docstrings Remove CLI Set defaults in function definition * Edit test video to 3 seconds from 35 Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it * Fix bug Co-authored-by: Noah Negrey <[email protected]> Co-authored-by: Leah E. Cole <[email protected]>
parthea
pushed a commit
to googleapis/google-cloud-python
that referenced
this pull request
Sep 22, 2023
…oudPlatform/python-docs-samples#2755) * Add auto-generated Logo Recognition Samples * Add tests Remove boilerplate Update copyright date Blacken Remove unused imports Shorten docstrings Remove CLI Set defaults in function definition * Edit test video to 3 seconds from 35 Reduce quota usage to allow for parallel tests Retain longer version only for tests that require it * Fix bug Co-authored-by: Noah Negrey <[email protected]> Co-authored-by: Leah E. Cole <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cherry-picked from tmp-generated-samples branch.
Add tests
Remove boilerplate
Update copyright date
Blacken
Remove unused imports
Shorten docstrings
Remove CLI
Set defaults in function definition
Edit test video to 3 seconds from 35