-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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 classification.ipynb #1080
Update classification.ipynb #1080
Conversation
Preview and run these notebook edits with Google Colab: Notebook diffs available on ReviewNB.com. |
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 What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@@ -22,7 +22,7 @@ | |||
"outputs": [], | |||
"source": [ | |||
"#@title Licensed under the Apache License, Version 2.0 (the \"License\");\n", | |||
"# you may not use this file except in compliance with the License.\n", | |||
"# You may not use this file except in compliance with the License.\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't change the license. This is how they apply it: https://www.apache.org/licenses/LICENSE-2.0#apply
@@ -960,7 +960,7 @@ | |||
}, | |||
"outputs": [], | |||
"source": [ | |||
"image_gen = ImageDataGenerator(rescale=1./255, zoom_range=0.5)" | |||
"image_gen = ImageDataGenerator(rescale=1./255, zoom_range=0.5) # zoom_range from 0 - 1 where 1 = 100" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is narrative doc, can you add this to the text section above to make this clearer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes sure
Corrected the licence text,
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
PiperOrigin-RevId: 279778860
No description provided.