Skip to content

Conversation

kaimo455
Copy link
Contributor

Change at line 468, follow the tf.image.resize, the new size format should be [IMG_HEIGHT, IMG_WIDTH]

Change at line 468, follow the tf.image.resize, the new size format should be [IMG_HEIGHT, IMG_WIDTH]
@googlebot
Copy link

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 with @googlebot 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 CLA has not been signed label May 17, 2020
@lamberta
Copy link
Member

Thanks. Before we can merge, can you please sign the CLA: https://cla.developers.google.com/

@kaimo455
Copy link
Contributor Author

Thanks. Before we can merge, can you please sign the CLA: https://cla.developers.google.com/

Signed.

@kaimo455
Copy link
Contributor Author

@googlebot I signed it!

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes CLA has been signed and removed cla: no CLA has not been signed labels May 19, 2020
@nemesszili
Copy link

Also discovered this a while ago, good catch!

Copy link
Member

@MarkDaoust MarkDaoust left a comment

Choose a reason for hiding this comment

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

Thanks!

@MarkDaoust
Copy link
Member

Darn square images...

@MarkDaoust MarkDaoust added the ready to pull Start merge process label May 20, 2020
@TensorFlow-Docs-Copybara TensorFlow-Docs-Copybara merged commit 84d1b1d into tensorflow:master Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA has been signed ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants