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

s4641465 #427

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

Conversation

BigGamerGary
Copy link

Topic number 5 implementation

@uqmarlonbran
Copy link
Collaborator

Training plot has strange convergence behaviour. Output also not as nice as other implementations.

@shakes76
Copy link
Owner

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

Adequate use and implementation (convergence strange) -1
Good spacing and comments
Header blocks missing -1

Recognition Problem

Solves problem (poor performance) -2
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting, could be more -1
No Data leakage
Difficulty: Normal -5

Commit Log

Meaningful commit messages, could be more informative) -1
Progressive commits used

Documentation

ReadMe OK, could be more descriptive, especially model -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 changed files for other students -2
Request Description minimal, needs more info -2

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