-
-
Notifications
You must be signed in to change notification settings - Fork 21
Update H2.md #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Update H2.md #48
Conversation
Added an example of trying to decode the meteo report message based on ADS-B ES MRAR
Bug ReportNo bugs found. Comments? Email us. |
WalkthroughA new example was added to the analysis section of the documentation, providing a detailed breakdown of a meteorological report from a specific flight. The example includes the raw data, a parsed version, and commentary on data components, markers, and format comparisons. Changes
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (5)
research/H2.md (5)
36-38
: Add a language identifier to the fenced code block
markdownlint
(MD040) flags this block because the opening fence (```
) lacks a language specifier.
Specify something generic liketext
(orplain
) to silence the linter and improve readability in rendered Markdown.-``` +```text
42-43
: Second fenced block also missing a language tagFor consistency with the first fix, add the same language identifier here.
-``` +```text
46-47
: Replace hard tabs with spaces to satisfy MD010The tab characters used for column alignment trip
markdownlint
rule no-hard-tabs. In Markdown code blocks you can safely use spaces without affecting the visual layout.
69-69
: Grammar tweak – “are transitions”, add commas & units formattingMinor stylistic/grammatical polish and clearer unit formatting.
-Worthy of note, altitude is given in tens. i.e. 583P = 5830ft. Guessing P and M is transition between Barometric altitude and flight level altitude, but I may be very wrong. +Worthy of note, altitude is given in tens (i.e., 583P = 5 830 ft). I am guessing P and M are transitions between barometric altitude and flight-level altitude, but I may be wrong.
71-71
: Insert missing determiner and tighten sentence-Hard to correlate given that this has no humidity data, and thus isn't on the MADIS AMDAR web viewer previously used for H1 AMDAR frame decoding +It is hard to correlate, given that the message has no humidity data; therefore it does not appear on the MADIS AMDAR web viewer previously used for H1 AMDAR frame decoding.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
research/H2.md
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
research/H2.md
[uncategorized] ~69-~69: You might be missing the article “the” here.
Context: ...i.e. 583P = 5830ft. Guessing P and M is transition between Barometric altitude and flight ...
(AI_EN_LECTOR_MISSING_DETERMINER_THE)
[uncategorized] ~72-~72: A determiner appears to be missing. Consider inserting it.
Context: ...ously used for H1 AMDAR frame decoding Data is similar to Mode-S/ADS-B ES [Meteorol...
(AI_EN_LECTOR_MISSING_DETERMINER)
🪛 markdownlint-cli2 (0.17.2)
research/H2.md
36-36: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
42-42: Fenced code blocks should have a language specified
null
(MD040, fenced-code-language)
43-43: Hard tabs
Column: 1
(MD010, no-hard-tabs)
46-46: Hard tabs
Column: 32
(MD010, no-hard-tabs)
46-46: Hard tabs
Column: 37
(MD010, no-hard-tabs)
46-46: Hard tabs
Column: 43
(MD010, no-hard-tabs)
Added an example of trying to decode the meteo report message based on ADS-B ES MRAR
Summary by CodeRabbit