-
Notifications
You must be signed in to change notification settings - Fork 178
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
Topic recognition #145
base: topic-recognition
Are you sure you want to change the base?
Topic recognition #145
Conversation
… with yolo architecture, just missing the testset
…hts if not present in repo
This is an initial inspection, no action is required at this point Difficulty: Normal Readme: Poor
Commit messages: OK Code:
Functionality/Performance:
|
…mentation for the original YOLOV1 concepts utilized in the YOLOV7 model
…is-2023 into topic-recognition
Thanks for the feedback, obviously a lot of stuff I didn't deliver on during the initial push. I have now done a series of changes, most related to the readme and documentation of the project. I hope this is sufficient. |
MarkingSome parts marked as per content for due date Good Practice (Design/Commenting, TF/Torch Usage)Adequate design and implementation, YOLOv7 external implementation -3 Recognition ProblemSolves problem Commit LogMeaningful commit messages DocumentationReadMe good but missing refs -1 Pull RequestSuccessful Pull Request (Working Algorithm Delivered on Time in Correct Branch) |
Cannot be merged as external implementation of YOLOv7, otherwise grade normally. |
Detection and classification of skin lesions on the ISIC2017 dataset using YOLOV7 architecture