demo: surface decoded RX PHY (bw/stbc/ldpc/sgi) in <devourer-stream>#104
Merged
Conversation
The <devourer-stream> line emitted only the rate index. Add the bandwidth, STBC, LDPC (FEC) and short-GI fields already present in RxAtrib so an external reference (e.g. an SDR-as-TX completeness harness) can assert the frame devourer received carries the PHY the transmitter encoded. Inserted before body= so the trailing hex-dump pattern downstream regex consumers key on is unchanged. Valid on 8812/8821; the 8814AU RX descriptor doesn't expose these at this offset, so they read as chip defaults there. tests/verify_stream_fields.sh: two-adapter loopback (no SDR) confirming the new fields appear on canonical-SA frames. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
<devourer-stream>RX line emitted only the rate index. This adds the bandwidth, STBC, LDPC (FEC) and short-GI fields (already inRxAtrib) so an external reference — e.g. an SDR-as-TX completeness harness — can assert that a received frame carries the PHY the transmitter actually encoded.body=, so the trailing hex-dump pattern downstream regex consumers key on is unchanged (same pass-through convention as the pre-Phy-level soft metrics on stream lines + BER-vs-SNR analyser #84 optional fields).tests/verify_stream_fields.sh: two-adapter loopback (no SDR) confirming the new fields appear on canonical-SA frames.Enables the SDR completeness work (josephnef/sdr2wifi) to validate the RX direction.
Generated with Claude Code