Skip to content

Conversation

@ywbrian
Copy link

@ywbrian ywbrian commented Nov 7, 2025

An Attention U-Net model that performs semantic segmentations of brain MRI scans from the OASIS dataset.

Create OASISDataset class to load dataset from files
- Add ConvBlock class to perform double convolution at encoder and decoder stages.
- Add AttentionBlock class to filter intermediate-scale features before merging skip connections.
Added a figure illustrating the Attention U-Net architecture and its source information.
Added DiceLoss and CombinedLoss classes for segmentation tasks.
Added dice coefficient calculation for evaluation and updated training epoch function.
Added main function to initialize datasets, loaders, and model.
Added a function to auto-detect label mapping, number of classes, and image size from segmentation masks. Updated NUM_CLASSES to be determined dynamically.
@wangzhaomxy
Copy link
Collaborator

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

File Organizing:
There is only a single Python script combining all functions, which does not meet the required instructions.

Problem Solving:

  • The algorithm appears to solve the problem correctly. However, the report contains no testing results, and there are no training, validation, or testing outcomes presented to demonstrate that the project has been fully completed.
  • Accuracy in testing dataset (Dice): NO test results in the report.

Model and functions:

  • It correctly uses PyTorch to construct the improved UNet 2D models and functions.
  • NO data augmentation.
  • Properly use the train/validation/test datasets.

Code design: Functional works.

Code comment and docstring:

  • Good code comments
  • Good function docstrings
  • NO header block

Difficulty: Easy.

Additional Comments:

  • Minimal commits, consisting mostly of generic messages such as ‘update’ or ‘add function.’ More meaningful and descriptive commit messages are expected.
  • The extremely minimal report, lacking any supporting evidence, indicates that the project is unfinished.

@wangzhaomxy
Copy link
Collaborator

Hi there, Please provide your student number or full name; otherwise, we won’t be able to allocate your marks.

@gayanku
Copy link
Collaborator

gayanku commented Nov 24, 2025

Marking

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

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

@ywbrian
Copy link
Author

ywbrian commented Nov 25, 2025

Hi there, Please provide your student number or full name; otherwise, we won’t be able to allocate your marks.

Check the bottom of the README

@wangzhaomxy wangzhaomxy removed the No_ID label Nov 27, 2025
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.

4 participants