Added Data Science Project: Brain tumor detector and classification#3
Open
anubhav9369 wants to merge 2 commits into
Open
Added Data Science Project: Brain tumor detector and classification#3anubhav9369 wants to merge 2 commits into
anubhav9369 wants to merge 2 commits into
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (300)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip Migrating from UI to YAML configuration.Use the |
Author
|
Good project for data science |
tkarim45
requested changes
Mar 28, 2026
tkarim45
left a comment
tkarim45
left a comment
Owner
There was a problem hiding this comment.
Thanks for the contribution! The brain tumor detection project looks interesting, but there are a few issues that need to be addressed before this can be merged:
1. Dataset images should not be committed to Git (~3,000 image files)
Committing the entire dataset directly into the repository will permanently bloat the repo size and make cloning very slow. Please:
- Remove the
Dataset 1/directory from the PR - Host the dataset externally (e.g., Kaggle, Google Drive, or Hugging Face)
- Add a download link and instructions in the README
2. Binary model file (brain_tumor_model.tflite)
Large binary files don't belong in Git history. Please:
- Remove
brain_tumor_model.tflitefrom the PR - Either provide a link to download it, or include instructions to train/generate it
3. README improvements
- Add instructions for how to download the dataset and model
- Include setup/installation steps (dependencies, Python version, etc.)
Once these changes are made, I'd be happy to review again!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Description
This PR adds a beginner-friendly data science project.
🚀 Features
🔧 Tech Stack
Python, Pandas, Scikit-learn
🙌 Contribution
This project is added as part of open-source contribution.****