From 16f23f36bbfd84c8821be1353ab6761aa85ef979 Mon Sep 17 00:00:00 2001 From: pgdalmeida Date: Tue, 23 Jun 2026 09:42:10 +0200 Subject: [PATCH] Refresh docs after the serial-protocol migration and hardware verification --- docs/known-issues.md | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/docs/known-issues.md b/docs/known-issues.md index 6c6e157..ef5366e 100644 --- a/docs/known-issues.md +++ b/docs/known-issues.md @@ -28,19 +28,19 @@ doctest unit-test suite (`ctest`). | # | Issue | Status | |---|-------|--------| | 13 | `[Motor]` degrees↔counts calibration | The `config.example.ini` values are **placeholders**. Calibrate `*_counts_per_deg` / `*_zero_count` against real `xenc` readings after homing on the rig. | -| 14 | Settle tolerance / timing | `kSettleTolCounts` (600) and the per-interval timing in [CaptureScheduler.cpp](../src/core/CaptureScheduler.cpp) are untested against hardware; tune against observed `ST` behaviour. | +| 14 | Capture sweep untested on hardware | Homing was verified live (see below), but the `MOVE → settle → trigger` sweep was **not** run with `--start`. `kSettleTolCounts` (600) and the per-interval timing in [CaptureScheduler.cpp](../src/core/CaptureScheduler.cpp) still need tuning against observed `ST` behaviour, alongside #13. | ## Verification caveats -- **Real MQTT wrapper** now compiles and links: `cmake -B build -DWITH_MQTT=ON` fetches and builds Paho - (C++ wrapper plus its bundled C library) from official Eclipse upstream and links `MqttControlChannel` - into the binary. Verified on GCC 16 / CMake 4 (see `cmake/Paho.cmake` for the toolchain-compat shims). -- **Real Serial/Vimba wrappers** were verified by code review, not compilation (no Vimba X SDK on the - development machine). They are faithful adaptations of the original code. First Vimba compile happens on - a machine with the SDK via `WITH_VIMBA=ON`. -- **Makefile parity** was never re-checked because the original Makefile build also can't run without the SDKs. - The Makefile has been removed in favour of CMake; if you need to confirm byte-for-byte behaviour, do a full - `WITH_VIMBA=ON WITH_MQTT=ON` build on a tower PC. +- **Full build verified on the LattePanda**: a `WITH_VIMBA=ON WITH_MQTT=ON` build compiles and links on the + device (real Vimba X SDK + Paho fetched). The MQTT wrapper also builds on the dev box (GCC 16 / CMake 4 — see + `cmake/Paho.cmake` for the toolchain-compat shims). +- **Serial protocol verified live against real firmware** (LattePanda, this session): `ENABLE`/`HOME`/`SPEED` + reach the firmware, the `ST` telemetry parses with zero unparsed lines across a full session, and a live + `--init` drove a clean re-home of both axes to `READY`. **Still unverified on hardware:** the capture sweep + (#14) and real-camera (Vimba) frame capture — earlier tests used `--mock-camera`. +- **Makefile parity** is moot: the Makefile was removed in favour of CMake, and the full + `WITH_VIMBA=ON WITH_MQTT=ON` CMake build now runs on the device. - **Demo mode** copies `bin/x64/Release/test_smoke.jxl`, resolved relative to the working directory. Run from a directory where that path exists, or extend `ImagePipeline::Params::demo_image`.