46974426 - pull request for merging 2d UNet into PatternAnalysis-2024 topic recognition branch

[robert123654]

Dear Shakes,

Please review my contribution to the PatternAnalysis-2024 repository and provide feedback if not appropriate for merging.

Thanks,

Robert

[yexincheng]

This is an initial inspection, no action is required at this point

1. Solves problem: The final DICE score on validaiton set is around 0.998 from images/50Epoc_validation_score.png. Training loss is provided. Prediction figures were not uploaded. Figures were not inserted into readme. Dependency and reference are not written. The report is a bit informal and hard to find the important information.
2. Implementation: Well-implemented functions and classes. Used ArgParse to set hyerparameters.
3. Design: Great
4. Comment: Sufficient
5. Difficulty: Easy

[gayanku]

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
	Good design and implementation.
	Spacing and comments.
	No Header blocks.	-1
Recognition Problem
	Good solution to problem.
	Driver Script present.
	File structure present.
	Good Usage & Demo & Visualisation & Data usage.
	Module present.
	Commenting present.
	No Data leakage found.
	Difficulty : Easy. UNet2D (Easy Difficulty)	-10
Commit Log
	Good Meaningful commit messages.
	Good Progressive commits.
Documentation
	Readme :Acceptable. Missing plots/figures. Report tone incorrect.	-4
	Model/technical explanation :Acceptable.	-2
	Description and Comments :Good. No references.
	Markdown used and PDF submitted. PDF checked.
Pull Request
	Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). Submitted on time.
	No Feedback required.
	Request Description is adequate.	-1
TOTAL		-18

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

[hanemma7moud]

Robert Slomka

[shakes76]

Likely data leakage with such high validation score -1