Skip to content

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