search

travisschriner / Lab3_FontController

0 stars 0 forks source link

Grading #1

Open travisschriner opened 10 years ago

travisschriner commented 10 years ago

Prelab was 50%

required functionality completed on time 3/3/14

toddbranch commented 10 years ago

Prelab - 5

Required - 40 B - 10 A - 0

Use of Git - Check Plus - 5 - Nice commits, bra! Your commit history makes it seem like you have B functionality or it's close - why didn't you get it checked off? Code Style - Check - 4 - Good comments to break code up. Some indentation issues. Almost everything in character_gen is indented one too far, while the signals are not indented. Magic numbers. README - Check - 17 - This is on the edge of a Check Minus. Give me an image showing how everything is connected together! Explain the purpose of each element. Put your debug notes into a separate Test / Debug section. Your code snippets are very useful, which is what's salvaging you a check - those are the most key code snippets in the lab.

Total - 81

travisschriner commented 10 years ago

Sir, I apologize but i forgot to comment that I did complete B functionality. Both you and Dr. York checked it off on time.