Skip to content

Add loading image from filename. #2778

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.

Sign up for GitHub

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

Jump to bottom
Merged
daspecster merged 1 commit into from
Dec 1, 2016
Merged

Add loading image from filename. #2778

daspecster merged 1 commit into from
Dec 1, 2016

Conversation

@daspecster
Copy link
Contributor

@daspecster daspecster commented Nov 30, 2016

Towards #2753

Adding support to pass in a filename when instantiating an Image.

@daspecster daspecster added the api: vision Issues related to the Cloud Vision API. label Nov 30, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 30, 2016
@daspecster daspecster mentioned this pull request Nov 30, 2016
Closed
10 tasks
dhermes
dhermes reviewed Nov 30, 2016
View reviewed changes
Copy link
Contributor

@dhermes dhermes left a comment

Choose a reason for hiding this comment

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

Mostly looks good (the benefit of tiny PRs!) though the constructor needs to be sorted out before I can sign off

docs/vision-usage.rst Outdated
>>> client = vision.Client(project='my-project', credentials=creds)
Creating an :class:`~google.cloud.vision.image.Image` object.

This comment was marked as spam.

Sign in to view

docs/vision-usage.rst Outdated
Creating an :class:`~google.cloud.vision.image.Image` object.
-------------------------------------------------------------

The :class:`~google.cloud.vision.image.Image` object is used to load your image

This comment was marked as spam.

Sign in to view

docs/vision-usage.rst Outdated
data from sources such as a Google Cloud Storage URI, raw bytes, or a file.


**From a Google Cloud Storage URI**

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

docs/vision-usage.rst Outdated
Image Properties
~~~~~~~~~~~~~~~~
----------------

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py Outdated
self._source = source_uri
else:
self._content = _bytes_to_unicode(b64encode(_to_bytes(content)))
if filename:

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py Outdated
with open(filename, 'rb') as file_obj:
content = file_obj.read()

if content:

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py
"""

def __init__(self, client, content=None, source_uri=None):
def __init__(self, client, content=None, filename=None, source_uri=None):

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

vision/tox.ini
pip install --quiet --upgrade {toxinidir}/../core
deps =
{toxinidir}/../core
mock

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

vision/unit_tests/test_image.py Outdated
image = self._make_one(CLIENT_MOCK, content=IMAGE_CONTENT)

_AS_DICT = {
_as_dict = {

This comment was marked as spam.

Sign in to view

vision/unit_tests/test_image.py Outdated
image = self._make_one(CLIENT_MOCK, source_uri=IMAGE_SOURCE)

_AS_DICT = {
_as_dict = {

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@daspecster daspecster mentioned this pull request Nov 30, 2016
Merged
dhermes
dhermes reviewed Dec 1, 2016
View reviewed changes
vision/google/cloud/vision/image.py
"""

def __init__(self, client, content=None, source_uri=None):
def __init__(self, client, content=None, filename=None, source_uri=None):

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py Outdated
def __init__(self, client, content=None, source_uri=None):
def __init__(self, client, content=None, filename=None, source_uri=None):
sources = set(source for source in (content, filename, source_uri)
if source is not None)

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py Outdated
if len(sources) != 1:
raise ValueError(
'Specify exactly one of \'content\', \'filename\', or'
'\'source_uri\'.')

This comment was marked as spam.

Sign in to view

vision/google/cloud/vision/image.py
self._content = _bytes_to_unicode(b64encode(_to_bytes(content)))
if filename is not None:
with open(filename, 'rb') as file_obj:
content = file_obj.read()

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

vision/tox.ini
pip install --quiet --upgrade {toxinidir}/../core
deps =
{toxinidir}/../core
mock

This comment was marked as spam.

Sign in to view

@daspecster
Copy link
Contributor Author

daspecster commented Dec 1, 2016

Thanks @dhermes! I cancelled travis and rerun after squashing.

@daspecster daspecster force-pushed the vision-add-filename-support branch from b77ed78 to 0bc1c4b Compare December 1, 2016 20:00
@daspecster daspecster merged commit e96c06a into googleapis:master Dec 1, 2016
@daspecster daspecster deleted the vision-add-filename-support branch December 1, 2016 20:23
@dhermes dhermes mentioned this pull request Dec 9, 2016
Merged
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
@daspecster
Merge pull request googleapis#2778 from daspecster/vision-add-filenam…
7a52ef3 
…e-support

Add loading image from filename.
parthea pushed a commit that referenced this pull request Oct 21, 2023
@daspecster
Merge pull request #2778 from daspecster/vision-add-filename-support
7804235 
Add loading image from filename.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@tseaver tseaver tseaver left review comments

@dhermes dhermes dhermes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@tseaver tseaver

Labels

api: vision Issues related to the Cloud Vision API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@daspecster @tseaver @dhermes @googlebot