lib/comms.py still contains the older JSON-over-UDP control sender that reads data/controls.json and sends to 127.0.0.1:5001 at a high rate.
Current app startup appears to use the newer binary bitmask control path instead, targeting the MCU on UDP 12345. The legacy JSON sender does not match the current MCU control protocol and could confuse future work.
Suggested cleanup:
- Confirm whether anything still imports or launches
lib/comms.py.
- If unused, remove it or move it behind clear legacy documentation.
- If still needed for a separate simulator/tool, rename/document it so it is not mistaken for the active MCU path.
- Keep docs aligned with the current
10.77.0.2 MCU network defaults.
lib/comms.pystill contains the older JSON-over-UDP control sender that readsdata/controls.jsonand sends to127.0.0.1:5001at a high rate.Current app startup appears to use the newer binary bitmask control path instead, targeting the MCU on UDP
12345. The legacy JSON sender does not match the current MCU control protocol and could confuse future work.Suggested cleanup:
lib/comms.py.10.77.0.2MCU network defaults.