Add documentation and script for detector setup #57
Conversation
|
|
||
| Please see the section that pertains to your operating system below. | ||
|
|
||
| ## Windows | ||
| ### Windows | ||
|
|
||
| Please install a recent version of Visual Studio Build Tools: https://visualstudio.microsoft.com/downloads/#build-tools-for-visual-studio-2022 |
There was a problem hiding this comment.
If this is needed then it indicates that there is a Python package that does not have wheels. Packages that do not have wheels are often lower quality, not well maintained, not widely used, so we should review if we really want to use such packages. Do you know which package does not have wheels?
There was a problem hiding this comment.
When I test on my system I believe it is wxpython and pyopengl-accelerate. The wxpython wheel in particular takes a very long time to build.
|
|
||
| This system is intended to be used over a LAN connecting Raspberry Pi -based detector(s) and a main computer. The Raspberry Pi's run the src.main_detector, while the main computer runs the src.main_pose_solver and optionally the src.gui.gui module. | ||
|
|
||
| To setup the Raspberry Pi -based detectors, run the setup/create_image.sh script on a compatible linux system. Flash the resulting image to a microSD card (or multiple) for use. |
There was a problem hiding this comment.
We cannot dictate what operating system developers should use. If it is absolutely impossible to perform these tasks on any other operating system, then provide instructions for running on docker.
But I suspect that instead of using bash, you could simply implement this in Python, making the project more uniform (all code implemented in Python) and making the code much more cleaner and easier to debug.
|
@keiranbarr you don't need to fix all the comments right now. If you want then you can create follow-up issues for comments that you don't want to deal with now, merge this PR, and we'll get back to those remaining issues later. |
keiranbarr
force-pushed
the
detector-image-creation-script
branch
from
a9ac0d2 to
f9dcc1a
Compare
Added additional documentation to doc/setup.md, as well as creating an image generation script for the detectors.
This was created in response to issue #56