Nanocockpit#156
Conversation
gemenerik
left a comment
gemenerik
left a comment
There was a problem hiding this comment.
Thanks. Couple of small comments.
| Start a bash console inside your docker: | ||
| ```sh | ||
| docker run --rm -it -v "${PWD}:/module/data/" -P -e "GAPY_OPENOCD_CABLE=interface/ftdi/olimex-arm-usb-tiny-h.cfg" --device-cgroup-rule="c 189:* rmw" -v /dev/bus:/dev/bus:ro -v /dev/serial:/dev/serial:ro bitcraze/aideck:3.8.1 | ||
| /bin/bash -c " |
There was a problem hiding this comment.
this is newline so treated as separate command
There was a problem hiding this comment.
would be nice if users can copy over as-is into their terminals
|
|
||
| ## Expected results | ||
|
|
||
| You can expect a framerate of ~30fps without, and ~60fps with disabling bidirectional CPX (change the CPX_SPI_BIDIRECTIONAL define in config.h for GAP8 and TODO (NINA changes?). |
There was a problem hiding this comment.
Leftover TODO. Are NINA changes required?
| Install the viewer as a Python package (best you do this in a virtual environment with python 3.9): | ||
|
|
||
| ```shell | ||
| $ pip install aideck_cpx_streamer |
There was a problem hiding this comment.
Remove $ to make copyable into terminal for direct execution, couple more of those below
| $ pip install aideck_cpx_streamer | |
| pip install aideck_cpx_streamer |
| // Himax auto-exposure | ||
| // - 0: disabled | ||
| // - 1: enabled [default] | ||
| #define HIMAX_AE (0) |
There was a problem hiding this comment.
Almost my full image clipped white in mildly sunny Sweden with the current defaults, but it also seemed that enabling auto exposure did not fix this for me. Have you had any success with auto-exposure?
OK with merging as is as long as we point to this file, s.t. users can change manual exposure settings.
|
|
||
| ## Expected results | ||
|
|
||
| You can expect a framerate of ~30fps without, and ~60fps with disabling bidirectional CPX (change the CPX_SPI_BIDIRECTIONAL define in config.h for GAP8 and TODO (NINA changes?). |
There was a problem hiding this comment.
Maybe config.h should be mentioned earlier on to make users aware that they can configure things there
2 participants
To include the nanocockpit framework in the bitcraze infrastructure, the gap8 code and python streamer are needed and added here in an example