Skip to content

Conversation

@CleoBriez
Copy link

This pull request attempts to process 2D HipMRI Prostate data to be trained by an Attention U-Net and then tested and visualised using predictive modeling. More details in the README file.

Please let me know if I seem to be missing something.

@wangzhaomxy
Copy link
Collaborator

There is no report and no experimental evidence provided. This is an unfinished project.

@CleoBriez
Copy link
Author

There is no report and no experimental evidence provided. This is an unfinished project.

I'm so sorry, I just realised I literally have 5 commits that didn't get added to the pull request.

is there anything I can do at this point?

@CleoBriez
Copy link
Author

I have the commits with timestamps. If there's anything I can do to add it in with the up-to-date changes. Please let me know

@CleoBriez
Copy link
Author

Hey, I'm so doubly sorry. I just check my files it's literally just that my commits past the first evidence I uploaded didn't get saved due to being 120MB and literally all the commits past that point just didn't get pushed.

I had the report uploaded to Blackboard at least. I can provide literally anything required if you're willing to take anything from what I had at time of submission.

@wangzhaomxy
Copy link
Collaborator

Hi there,

Sorry, this situation is beyond the scope of what a tutor can handle. My suggestion would be to reach out to your course coordinator Shakes to see if there’s anything that can be done.

Zhao

@CleoBriez
Copy link
Author

I think I have managed to fix the merge error I was having and have reached out to EdStem as well as Shakes

@CleoBriez
Copy link
Author

The file that was too large has been removed and linked here: https://1drv.ms/u/c/3c907cdb67f98515/ERAyUneJKppAiv3ZthRVG98BJLXx4qc_Hy7mGyG2szSbZQ?e=PMMwEL

@gayanku gayanku added the help wanted Extra attention is needed label Nov 24, 2025
@gayanku
Copy link
Collaborator

gayanku commented Nov 24, 2025

@shakes76

@wangzhaomxy
Copy link
Collaborator

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

File Organizing: Well-organized files.

Problem Solving:

  • The algorithm solves the problem appropriately. However, no report and evidence supporting a successful experiment.
  • Accuracy in testing dataset (Dice): No results reported.

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: Good.

Code comment and docstring:

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

Difficulty: Normal.

Additional Comments:

  • Good commits
  • The overly simplified report indicates that the project was ultimately unsuccessful

@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.
Header blocks.
Recognition Problem
OK solution to problem. -3
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found.
Difficulty : Normal. Normal -ImprovedUnet2D-5
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Acceptable. No results-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).
Feedback action require: Feedback marks possible +2 if the requested changes are made. Remove cache for merge.-2
Request Description is adequate. -1
TOTAL-17

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

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.

5 participants