Skip to content

Conversation

@wakameds
Copy link

ID: s46331391
Task: Improved UNet

The pull request includes the seven files

  • README.md
  • criterion.py
  • dataloader.py
  • driver.py
  • model.py
  • model_train_val.py
  • visualise.py

README.md provides a brief explanation about the model structure, model evaluation with dice coefficient, dice loss, and image segmentation. The dice coefficient of the test images shows over 0.87.

wakameds and others added 30 commits September 24, 2021 14:54
remove png file
remove png file
@wakameds
Copy link
Author

Removed commit 02e5662

@wakameds wakameds requested a review from wooboyeong November 17, 2021 11:15
@wakameds
Copy link
Author

Hi Professor. I think the conflict problem happened due to a response to remove a commit log request. If possible, would you tell me what I should do to solve the problem?

@shakes76
Copy link
Owner

TF/Torch Usage

Adequate use and implementation

Good Practice (Design/Commenting)

Good spacing and comments
Header blocks

Algorithm

Driver Script present
Shows Usage & Demo & Visualisation & Data usage
Module present
No Data leakage
Difficulty: Easy -10

Commit Log

Meaningful commit messages
Progressive commits used

Documentation

Good Description and Comments
Markdown used PDF submitted

Pull Request

Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
Still need to fix the merge conflict before I can merge -2
Request Description OK

@wakameds
Copy link
Author

Hi, Professor Shakes. I fixed the merge conflict. Please confirm it.

@shakes76
Copy link
Owner

Feedback marks awarded.

@shakes76 shakes76 merged commit 018b724 into shakes76:topic-recognition Nov 23, 2021
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