Skip to content
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

Detection of Skin Lesions using a YOLO-based Detection Network #193

Open
wants to merge 28 commits into
base: topic-recognition
Choose a base branch
from

Conversation

mermalade0325
Copy link

@mermalade0325 mermalade0325 commented Nov 11, 2024

This pull request aims to address the problem of detecting skin lesions within the ISIC 2017/2018 dataset. The primary goal is to develop a robust detection and classification network that can accurately identify lesions in medical images, ensuring each detection has a minimum Intersection Over Union (IoU) of 0.8 on the test set, thus achieving a suitable level of accuracy for classification. The project utilizes a YOLO-based detection network (YOLOv7).

Problem Statement
Skin cancer detection relies on identifying lesions accurately in medical images. This project contributes by building a deep learning model focused on detecting lesion locations without classifying lesion types. The main goals are:

  • Localization: Detect lesion locations with high accuracy.
  • Performance Metrics: Achieve a minimum IoU of 0.8 for lesion detection on the test set.

Files and Structure

The project is organized into the following files:

  • modules.py: Contains core components of the model, structured as classes or functions for easy reuse and extension.
  • dataset.py: Implements the data loader for the ISIC dataset, with preprocessing functions.
  • train.py: Contains training, validation, and testing routines. It imports the model from modules.py and data loader from dataset.py. Training metrics, such as loss, are plotted and saved to help track model performance.
  • predict.py: Shows how to load the trained model for predictions. Outputs include visualizations with bounding boxes around detected lesions.
  • README.md: Documents the project setup, usage, and module descriptions.

Dataset

Regarding the data downloaded from ISIC, it should be organised as follows, with labels generated from dataset.py: COMP3710_YOLO

An appendix in the README file links to the ISIC dataset for reference.

As requested by tutors, this is a new Pull Request submitted from the correct branch.

Testing branch
Reupload of files to the correct branch (topic-recognition)
Adjusting folder structure, deleting this file
Adjusting folder structure, deleting this file
Creating new ReadMe in correct directory
Moving ReadMe work into this new file with the correct directory and branch as specified
@hanemma7moud hanemma7moud added _YOLO PDF PDF submitted labels Nov 13, 2024
@hanemma7moud
Copy link
Collaborator

hanemma7moud commented Nov 13, 2024

@shaivikaaaa @gayanku

@shaivikaaaa
Copy link
Collaborator

Hi @mermalade0325
I believe you already have 1 PR - which is open and have been provided Feedback for
I am not sure why is there another one

@mermalade0325
Copy link
Author

mermalade0325 commented Nov 13, 2024

Hi @shaivikaaaa,
Tutors provided feedback on the other PR mentioning that I accidentally merged from my main instead of my topic_recognition branch. As per their feedback I made a new PR from the correct branch, and implemented all the edits they asked for on the read_me, results images, PR details etc. on here! Hope this is okay!

@gayanku gayanku added help wanted Extra attention is needed duplicate This issue or pull request already exists and removed help wanted Extra attention is needed labels Nov 13, 2024
@gayanku
Copy link
Collaborator

gayanku commented Nov 14, 2024

Duplicate is #188. Please fix 188 as per feedback required section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discard duplicate This issue or pull request already exists PDF PDF submitted _YOLO
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants