hnen / NumberClassifier

0 stars 0 forks source link

Peer review #2

Open armijuha opened 2 years ago

armijuha commented 2 years ago

ZIP package was loaded Saturday 27.8. about 2 PM.

Couldn't open with Netbeans but the problem is on my side, it seems that gradle programs are not working with my Netbeans version. Anyway everything was working fine with the Visual Studio (well some error because I had too old Java version I quess but still). Program looks nice and works very well. I try all the numbers and it recognized every single one even that my handwriting was not too pretty. Why does it think that number is "4" before I draw anything? :) All but one of the tests (32/33) was passed and that one failed because my too old Java version.

To me this program seems very professional and impressive. I don't have enough time to understand it deeply right now but everything looks great and coding style is clear to read. Machine learning is interesting so maybe I'll try to come back to this if I have more time later. Problem for me is that it's pretty impossible to give you any advice for this, I just don't have the skills for it.

Great documentations also, very comprehensive and clear. Didn't even spot spelling errors. Just a great job!