-
Notifications
You must be signed in to change notification settings - Fork 6.5k
[video] Streaming AutoML Classification #2313
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
Changes from all commits
b6c7a5b
9c214db
a1ed8e4
f2a052f
ec58563
f1298a1
c02d79c
bae60dc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
#!/usr/bin/env python | ||
|
||
# Copyright 2017 Google, Inc | ||
# Copyright 2019 Google, LLC | ||
# | ||
# Licensed under the Apache License, Version 2.0 (the "License"); | ||
# you may not use this file except in compliance with the License. | ||
|
@@ -16,6 +16,7 @@ | |
|
||
from six.moves.urllib.request import urlopen | ||
import time | ||
import os | ||
import uuid | ||
|
||
import beta_snippets | ||
|
@@ -160,3 +161,13 @@ def test_track_objects_gcs(): | |
assert text_exists | ||
assert object_annotations[0].frames[0].normalized_bounding_box.left >= 0.0 | ||
assert object_annotations[0].frames[0].normalized_bounding_box.left <= 1.0 | ||
|
||
|
||
@pytest.mark.slow | ||
def test_streaming_automl_classification(capsys, in_file): | ||
project_id = os.environ['GCLOUD_PROJECT'] | ||
model_id = 'VCN6363999689846554624' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @anguillanneuf, responding here, but for But for
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This still shows a literal model_id identifier here, I don't see it updated to use an environment variable There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry I'm late to the party here but I just noticed that in_file isn't defined in this test - it's defined in test_track_objects_gcs() - should its definition also be added to this function so that we're following the practice of making sure tests can run alone and/or in any order? @anguillanneuf There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks Leah for catching this! I had meant to use 'video_path' instead of 'in_file' because 'video_path' can be used as a fixture. Addressing it now. |
||
beta_snippets.streaming_automl_classification( | ||
in_file, project_id, model_id) | ||
out, _ = capsys.readouterr() | ||
assert 'brush_hair' in out |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
google-cloud-videointelligence==1.8.0 | ||
google-cloud-videointelligence==1.11.0 | ||
google-cloud-storage==1.14.0 |
Uh oh!
There was an error while loading. Please reload this page.