abhisheks008 / DL-Simplified

Deep Learning Simplified is an Open-source repository, containing beginner to advance level deep learning projects for the contributors, who are willing to start their journey in Deep Learning. Devfolio URL, https://devfolio.co/projects/deep-learning-simplified-f013
https://quine.sh/repo/abhisheks008-DL-Simplified-499023976
MIT License
357 stars 299 forks source link

[Feature Addition]: Web App for Dog Breeds Classification #590

Closed Himanshu8850 closed 4 months ago

Himanshu8850 commented 4 months ago

Pull Request for DL-Simplified 💡

Issue Title :

Closes: #551

Describe the add-ons or changes you've made 📃

-Added a web-app folder -Added requirements.txt -Added code for saving best model in .keras file in ResNet50v2(98%).ipynb

Type of change ☑️

What sort of change have you made:

How Has This Been Tested? ⚙️

I have tested my web-app on my browser by checking real-time classification and output on page on image input.

Checklist: ☑️

github-actions[bot] commented 4 months ago

Our team will soon review your PR. Thanks @Himanshu8850 :)

abhisheks008 commented 4 months ago

Can you just drag and drop the demo video from your local system to README file, so that it can be viewed directly in the README.

@Himanshu8850

abhisheks008 commented 4 months ago

It should be like this, https://github.com/abhisheks008/DL-Simplified/tree/main/Brain%20Tumor%20Detection/Web%20App

Himanshu8850 commented 4 months ago

is the readme ok now?

abhisheks008 commented 4 months ago

The Web App's README file should be inside the Web App folder. You have merged both the README files together, that's not the correct way to do it. Please have a look at the reference project carefully and update the README properly.

Also the video is not showing in the README. Just copy the demo video file from using ctrl+c and paste it in the README by ctrl+v, no need to hyperlink the url of the uploaded video.

@Himanshu8850

Himanshu8850 commented 4 months ago

Done with the changes, please review them.

Himanshu8850 commented 4 months ago

Sir, is the issue resolved?