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

Spelling & Typo Corrections #12

Merged
merged 19 commits into from
Oct 4, 2022
Merged

Spelling & Typo Corrections #12

merged 19 commits into from
Oct 4, 2022

Conversation

mikames
Copy link
Contributor

@mikames mikames commented Sep 23, 2022

Issue #, if available:

Description of changes:
addressed several spelling and typos in the notebooks.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

fixed typo should be 262 images not 1000
fixed spelling typos
Copy link
Member

@matteofigus matteofigus left a comment

Choose a reason for hiding this comment

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

Added a couple of questions

removed collection name
small corrections.
supporting images for lab 3.2
@mikames mikames requested a review from matteofigus September 29, 2022 15:12
Copy link
Member

@matteofigus matteofigus left a comment

Choose a reason for hiding this comment

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

A couple of changes needed. I assume we are allowed to use these 4 new photos?

mikames and others added 8 commits September 29, 2022 11:19
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…Faces and SearchFacesByImage.ipynb

Co-authored-by: Matteo Figus <[email protected]>
…etup S3 buckets for IDV modules.ipynb

Co-authored-by: Matteo Figus <[email protected]>
@mikames mikames requested a review from matteofigus September 29, 2022 18:29
@matteofigus matteofigus merged commit 5fbab47 into aws-samples:main Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants