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

s46300511 Lesion Segmentation using Improved UNET Pull Request Updated #181

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

Conversation

ChevScore
Copy link

Updated pull request with following fixed:

  • Missing image display in README.md
  • Wrong file structure
  • Proof of running

@shakes76
Copy link
Owner

Previously marked as #179

@shakes76
Copy link
Owner

Marking

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness

Good Practice (Design/Commenting, TF/Torch Usage)

Adequate design and implementation
Good spacing and comments
Header blocks missing -1

Recognition Problem

Solves problem but no predicted mask outputs or DSC results shown for test set -2
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage, initially no results or plots -1
Module present
Commenting minimal -1
No Data leakage
Difficulty: Easy -10

Commit Log

Meaningful commit messages minimal -1
Progressive commits used, very few used -1

Documentation

ReadMe OK, no usage -1, no refs -1
Model/technical explanation minimal -1
Good Description and Comments
Markdown used and PDF submitted

Pull Request

Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required, changes done -2 +2
Request Description minimal -1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants