server/REVIEW.md

13 lines
1.2 KiB
Markdown
Raw Permalink Normal View History

2020-05-15 21:44:00 -07:00
# Code Review Changes
2020-05-15 21:56:41 -07:00
Original tables with verbatim feedback can be found in [this Google Doc](https://docs.google.com/document/d/19d5U-ieQyGVVNQjj0rUouzIqF53evEP6yss-wyqNXiQ/edit?usp=sharing).
2020-05-15 21:44:00 -07:00
|Comment|Adjustment|
|-------|----------|
|The `models.py` file is too difficult to understand.| `models.py` updated with Python docstrings.|
|The code in general is not commented.| Every function in the codebase commented with Python docstring to explain usage and implementation. Some comments added to functions longer than a few lines.|
|It's difficult to set up the server software.| `requirements.txt` file updated with all packages; `default.nix` provided via `README.md` to provide alternative installation method.|
|No README file.| README file added with installation and configuration instructions.|
|No unit tests.| Developed [cow description language](https://dev.danilafe.com.CS-46x/cdl.git) to test database access (tests still need to be run manually, due to time constraints).
|Unrolled list of variables can be converted to list.| Unrolled fields kept to maintain isomorphism with hardware, where unrolling variables is necessary due to hardware and radio limitations. Methods written to limit the need for manually accessing `coord*x` and `coord*y` variables.|