Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Topic-Recognition, VQ VAE (OASIS Dataset) #461

Open
wants to merge 30 commits into
base: topic-recognition
Choose a base branch
from

Conversation

Anwealso
Copy link

PatternFlow project submission for Alex Nicholson (s4531620). This project implements a VQ-VAE network paired with a PixelCNN generator network trained together on the OASIS brain MRI image datset for use in generating novel and realistic synthetic brain MRI images. Instructions for running the code and demonstration of outputs, as well as a deeper overview of how it works are available in the included README.md file.

@SiyuLiu0329
Copy link
Collaborator

SiyuLiu0329 commented Nov 9, 2022

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

  • Recon: Good
  • Generative model: Good
  • Documentation: Good
  • Code and results: Good
  • Commits: Good

@shakes76
Copy link
Owner

Good Practice (Design/Commenting, TF/Torch Usage)

Adequate use and implementation
Good spacing and comments
Header blocks

Recognition Problem

Solves problem (generations poor quality) -1
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Hard

Commit Log

Meaningful commit messages
Progressive commits used

Documentation

ReadMe acceptable, but not architecture diagram -1
Good Description and Comments
Markdown used PDF submitted

Pull Request

Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
Feedback required, revert changes to other student files -2
Request Description OK, could have more info -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.

4 participants