Skip to content

feat(comms):JSON_and_NMEA_GGA_receive_parsers - #11

Merged
Onlyti merged 3 commits into
mainfrom
feat/comms-parsers
Sep 21, 2026
Merged

Onlyti merged 3 commits into
mainfrom
feat/comms-parsers

Conversation

@Onlyti

@Onlyti Onlyti commented Sep 21, 2026

Copy link
Copy Markdown
Owner

Scope

  • add receive-only parser for the existing compact JSON state wire format
  • add receive-only parser for the existing NMEA 0183 GPGGA wire format
  • leave comms_templates.hpp TX encoders and realtime control routing unchanged

Validation

  • new parser tests: 10
  • targeted comms tests: 43/43
  • validation preset: 502/502 locally
  • docs/VALIDATION.md is PENDING because tests/unit/CMakeLists.txt changed

Rejection evidence

nmea_gga: checksum mismatch (expected 47, got 00)

Canonical promotion requires three successful CI attempts on this exact head.

Canonical CI run 35630925124 attempts 1/2/3 all green on the same head
(5 jobs each, 504/504, cmake 3.31.10).
@Onlyti
Onlyti merged commit 8f9f943 into main Sep 21, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant