Update SA.md with character code explanations

[SoTi-AF]

Added explanations for various character codes used in the data format.

Summary by CodeRabbit

• Documentation
  • Clarified documentation by adding explicit mappings for status letters (V, 2, S, H, E) after the time field in examples and analysis, making trailing-code meanings and valid values easier to interpret.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f7653fc and 1f84be0.

📒 Files selected for processing (1)

• research/SA.md

🚧 Files skipped from review as they are similar to previous changes (1)

• research/SA.md

---

📝 Walkthrough

Walkthrough

Adds explicit mappings for status letters appearing after the time field in research/SA.md, defining five codes (V, 2, S, H, E) in both the Examples/Notes and Analysis sections of the document.

Changes

Cohort / File(s)	Summary
Documentation research/SA.md	Inserted explicit code mappings for status letters after the time field: V = VDL-M0/A (Plain Old ACARS), 2 = VDL-M2, S = SATCOM, H = HFDL, E = unknown. Added entries in both Examples/Notes and Analysis sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• makrsmark

Poem

🐰 I nudged the docs with fuzzy paws and glee,
V, 2, S, H, E now hop for all to see.
Codes once hidden, now clear in the light,
A little rabbit reads them—what a sight!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Update SA.md with character code explanations' directly and accurately describes the main change: adding character code mappings to SA.md documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@research/SA.md`:
- Around line 68-72: The mappings for H and E are unresolved and conflict with
the “all current examples” set; update the document by marking H and E as
tentative (e.g., add “(tentative)” after H = HFDL and E = ? or move them into a
new “Unconfirmed mappings” section) or, if you have observed examples, add those
explicit examples showing H and/or E after the time field and remove the
tentative flag; ensure you reference the mapping symbols H and E and preserve
the existing V/2/S entries unchanged.

---

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cc076bd and f7653fc.

📒 Files selected for processing (1)

• research/SA.md

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Clarify evidence level for H and E mappings.

These mappings conflict with Line 66’s “all current examples” set and E = ? is still unresolved. Please label H/E as tentative (or move them to a separate “unconfirmed” list), or update Line 66 with observed examples that include them after the time field.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@research/SA.md` around lines 68 - 72, The mappings for H and E are unresolved
and conflict with the “all current examples” set; update the document by marking
H and E as tentative (e.g., add “(tentative)” after H = HFDL and E = ? or move
them into a new “Unconfirmed mappings” section) or, if you have observed
examples, add those explicit examples showing H and/or E after the time field
and remove the tentative flag; ensure you reference the mapping symbols H and E
and preserve the existing V/2/S entries unchanged.

[SoTi-AF]

E = remains unknown