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

Update images.ipynb #761

Merged
merged 1 commit into from Jul 8, 2019

Conversation

Projects
None yet
6 participants
@fabiofumarola
Copy link
Contributor

commented Jul 7, 2019

fixed Attribute error in tf.read_file. It is tf.io.read_file

Update images.ipynb
fixed Attribute error in `tf.read_file`. It is `tf.io.read_file`
@googlebot

This comment has been minimized.

Copy link

commented Jul 7, 2019

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

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no label Jul 7, 2019

@tfdocsbot

This comment has been minimized.

Copy link
Collaborator

commented Jul 7, 2019

Preview and run these notebook edits with Google Colab:

Notebook diffs available on ReviewNB.com.
@lamberta
Copy link
Member

left a comment

Thanks.
We're focused on TF2 docs now and looks like this has been fixed there: https://github.com/tensorflow/docs/blob/master/site/en/r2/tutorials/load_data/images.ipynb

No harm merging this but I wouldn't spend much time on the TF1 docs. Thanks

@googlebot

This comment has been minimized.

Copy link

commented Jul 8, 2019

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@lamberta

This comment has been minimized.

Copy link
Member

commented Jul 8, 2019

Confirmed CLA for username.

@TensorFlow-Docs-Copybara TensorFlow-Docs-Copybara merged commit 83543e7 into tensorflow:master Jul 8, 2019

2 of 3 checks passed

Ubuntu Sanity Check Internal CI build failed
Details
cla/google CLAs have been manually verified by Googler who has set the 'cla: yes' label
import/copybara Change imported to the internal review system
Details

TensorFlow-Docs-Copybara pushed a commit that referenced this pull request Jul 8, 2019

Copybara-Service
Merge pull request #761 from fabiofumarola:patch-1
PiperOrigin-RevId: 257034334
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.