leonjia0112 / ec601_passenger_screening

1 stars 1 forks source link

Code Review 2 - Shruthi #15

Open ShruthiSiva opened 6 years ago

ShruthiSiva commented 6 years ago

hash.py and data.sql:

Although the code was simple enough to understand, I would suggest having a few lines of comment describing where this module fits in your entire project and what the code does. I'm not sure I'd have been able to understand the hash.py part of not for EC602.

augmentation.py:

The code is really well described in the comments and is easy to read and understand for someone who's fairly new to machine learning. I also like how the program is interactive and gives the user a heads up at each stage.

VGGNet.py:

Specifying the input parameters and the return parameters of the function at the start of your code helped me understand what this module does.

I think it would really help if you guys described where each module fits in into your project so it would help readers who aren't very familiar with the topic to understand it better,

Rest assured you guys have put together a great project! Congratulations:)