search

weishiyan / felinemotion

This is the felinemotion repo for CSE583 project
MIT License
1 stars 3 forks source link

Feedback on final presentation #6

Open joseph-hellerstein opened 4 years ago

joseph-hellerstein commented 4 years ago
  1. Component specification doesn’t list components.
  2. 10 python modules; only 2 test files.
  3. cruft in repository (checkpoint for ipynb) - .gitignore is not fully setup
  4. No example? Readme.md has instructions but they aren’t well formatted, no screen shot or other example.
  5. Functional specification incomplete
  6. Still pep8 non-compliant
ng-git commented 4 years ago

Hello Professor,

We have fixed our repo, documentation and code according to your feedback. We also want to to let you know that since we were using cv2 package, which is from a third party, pylint will always show it as an error. Therefore, we had to make exception in pylint for cv2.

Thank you, Minh Nguyen / Felinemotion Team