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

Face Recognition #110

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Face Recognition #110

wants to merge 10 commits into from

Conversation

Dhanush2612
Copy link

Python programs to capture a user's face and generate dataset. Then, detect and recognize user's face by using opencv.
Readme and sample output has been included in this PR.

Python programs to capture a user's face and generate dataset. Then, detect and recognize user's face by using opencv.
Readme and sample output has been included in this PR.
@Dhanush2612
Copy link
Author

@geekquad
This is the PR for issue #46

@Dhanush2612 Dhanush2612 mentioned this pull request Mar 30, 2021
Copy link
Owner

@geekquad geekquad left a comment

Choose a reason for hiding this comment

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

Hey @Dhanush2612,
Please go through the Contributing Guidelines. We are accepting implementations only in a .ipython Notebook. Please make the necessary changes.

@Dhanush2612
Copy link
Author

Sure

@Dhanush2612
Copy link
Author

@geekquad
I have made the changes.

@Dhanush2612 Dhanush2612 mentioned this pull request Apr 2, 2021
Copy link
Owner

@geekquad geekquad left a comment

Choose a reason for hiding this comment

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

Hey @Dhanush2612,
image

The structure looks messy and there Is no description as to which file is achieving what task.
Please do the needful.

@Dhanush2612
Copy link
Author

Ok

@Dhanush2612
Copy link
Author

@geekquad
I have made changes as per your request.
I have added each file to various folders depending on their operations.

@Dhanush2612
Copy link
Author

@geekquad
Refer to last 6 commits above from
6a9ab92 to 832111d

@geekquad
Copy link
Owner

Hey @Dhanush2612, files changed show no difference.
Have a look here.

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.

2 participants