Skip to content

Conversation

rabinadk1
Copy link
Contributor

Previously it was [0,1] but the input mobilenetV2 needs it in the form of [-1,1]. So made changes in the notebook to accommodate the change.

Previously it was [0,1] but the input mobilenetV2 needs it in the form of [-1,1]. So made changes in the notebook to accommodate the change.
@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 5, 2020
@tfdocsbot
Copy link
Collaborator

Preview

Preview and run these notebook edits with Google Colab:

Diffs

For cleaner diffs (and easier review), format notebooks with the nbfmt.py tool:
./tools/nbfmt.py --ignore_warn path/to/notebook.ipynb
Rendered notebook diffs available on ReviewNB.com.

@rabinadk1
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 5, 2020
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!

@tfdocsbot tfdocsbot added the lgtm Community-added approval label May 5, 2020
@MarkDaoust MarkDaoust added the ready to pull Start merge process label May 5, 2020
@TensorFlow-Docs-Copybara TensorFlow-Docs-Copybara merged commit 3fdb62f into tensorflow:master May 14, 2020
@rabinadk1 rabinadk1 deleted the patch-1 branch May 15, 2020 11:23
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 lgtm Community-added approval ready to pull Start merge process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants