-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
[Model Enhancement]: Brain Tumor Detection with Web Application #542
Conversation
All the files were Unnecessary. Might help to reduce the size of the Repository. Repository Optimization.
Unnecessary Files. Not needed. Might help to reduce the size of the repository.
Made A README.md for this sub project.
Made Changes to the list.
Made Minor Changes to the Documentation.
Changed some Mistakes.
Wrote the Details on how the model has been done!!
Removed Unnecessary File.
Removed Unnecessary file.
Adding the Jupyter Notebook.
Remove Useless Files
Made This by Mistake
Our team will soon review your PR. Thanks @TheNaiveSamosa :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Please include the
Dataset
folder, and inside that create aREADME
to mention the source of the dataset. - Include the
Images
folder and insert all the EDA results inside that. - Rename the
Deployment
folder toWeb App
. - Please follow the separate
README templates
for the main README file and for the web app's README file. Main README template is here and Web App README template is here. - Upload the video inside the
Web App
folder. - Use at least 2-3 algorithms/models for this project instead of going with a single model. My point is to make a comparison of the models to find out the best fitted one for this project as well as for the web app deployment.
Need to fix these changes.
@TheNaiveSamosa
Take some time, why are you in a hurry for pushing the codes. It's a 3-months long open source event, don't be in a rush. @TheNaiveSamosa |
Changes Implimented. |
Hi @TheNaiveSamosa sorry to bother you again. Instead of putting the Google drive link, why don't you just upload the video in the |
Acutally I tried uploading the video to the |
Hi @TheNaiveSamosa use this website and compress the video https://www.freeconvert.com/video-compressor. GitHub allows you to upload file of maximum 25 MB. |
Hi @TheNaiveSamosa try this markdown trick, |
Yeee!!! It's working 🎉 Let me have a final check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your PR is approved and ready to be merged. You did a fantastic job. Really appreciated your efforts for this issue.
@TheNaiveSamosa
Thanks, Great working with you. |
Pull Request for DL-Simplified 💡
Issue Title : [Model Enhancement]: Brain Tumor Detection with Web Application #531
Info about the related issue (Aim of the project) : The existing brain tumor detection project has been implemented on a small dataset with suboptimal accuracy and recall rates. The project is also missing an app.py file.
Name: Aditya Khamitkar
GitHub ID: TheNaiveSamosa
Email ID: thenaivesamosa@gmail.com
Identify yourself: Contributor for GirlScript Summer of Code
Closes: #Brain Tumor Detection Improvement Task
Describe the add-ons or changes you've made 📃
Enhanced the accuracy and recall rates of the brain tumor detection model by implementing the following changes:
Type of change ☑️
How Has This Been Tested? ⚙️
Tested the functionality by running the Flask application locally and uploading test images to ensure accurate predictions.
Checklist: ☑️
Please review and merge this PR. Thank you!