-
Notifications
You must be signed in to change notification settings - Fork 356
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
Facerecognition: updated deepface.ipynb #721
Facerecognition: updated deepface.ipynb #721
Conversation
@invigorzz313 ofcourse I ran it. And attached the output of every cell in the #645 mentioned pdf. also attaching the same here. |
There seems to be an issue then. The PDF you shared has additional cells but your commits in this PR has missing cells after the one in the above shared screenshot. |
@invigorzz313 I've updated the deepface.ipynb file so could please check that once. And anyway I'll update it once again. |
Did you push those changes after updating? Because I notice no new changes since last. |
@invigorzz313 My bad, may be I didn't push the changes I guess now you can check the file. |
@Raghucharan16 use matplotlib or any other libraray to show image used in the project and proper comments in your code , add results in your readme.md file. |
@invigorzz313 notebook and readme has been updated accordingly. check it out once. |
this solves #630 and new version of #645