Niketkumardheeryan / ML-CaPsule

ML-capsule is a Project for beginners and experienced data science Enthusiasts who don't have a mentor or guidance and wish to learn Machine learning. Using our repo they can learn ML, DL, and many related technologies with different real-world projects and become Interview ready.
MIT License
377 stars 317 forks source link

Facerecognition: updated deepface.ipynb #721

Closed Raghucharan16 closed 3 months ago

Raghucharan16 commented 4 months ago

this solves #630 and new version of #645

invigorzz313 commented 4 months ago

You need to run all the cells for the proper output.

image

Raghucharan16 commented 4 months ago

@invigorzz313 ofcourse I ran it. And attached the output of every cell in the #645 mentioned pdf. also attaching the same here. deepface.pdf

invigorzz313 commented 4 months ago

@invigorzz313 ofcourse I ran it. And attached the output of every cell in the #645 mentioned pdf. also attaching the same here. deepface.pdf

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. You can check for yourself in the above 'Files changed' tab.

Raghucharan16 commented 4 months ago

@invigorzz313 I've updated the deepface.ipynb file so could please check that once. And anyway I'll update it once again.

invigorzz313 commented 4 months ago

@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.

Raghucharan16 commented 4 months ago

@invigorzz313 My bad, may be I didn't push the changes I guess now you can check the file.

Niketkumardheeryan commented 4 months ago

@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.

Raghucharan16 commented 4 months ago

@invigorzz313 notebook and readme has been updated accordingly. check it out once.