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
Pr param high latency freq #23048
Pr param high latency freq #23048
Conversation
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.
Nice work. I would have done it as a single parameter, but this is fine too.
src/modules/mavlink/module.yaml
Outdated
|
||
MAV_${i}_HL_FREQ: | ||
description: | ||
short: Configures the frequency of HIGH_LATENCY2 stream for iridium instance ${i} |
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.
short: Configures the frequency of HIGH_LATENCY2 stream for iridium instance ${i} | |
short: Configures the frequency of HIGH_LATENCY2 stream for instance ${i} |
src/modules/mavlink/mavlink_main.cpp
Outdated
err_flag = true; | ||
|
||
} else { | ||
if (freq >= 0) { |
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.
if (freq >= 0) { | |
if (freq >= 0.f) { |
0b80479
to
2b5038c
Compare
Solved Problem
Missing parameter to configure the frequency of the
HIGH_LATENCY2
stream iniridium
mode.Solution
MAV_${i}_HL_FREQ
to configureHIGH_LATENCY2
stream frequency iniridium
mode.mavlink_main.cpp
, added parsing of cli option-F
setting theHIGH_LATENCY2
stream frequency iniridium
mode.Changelog Entry
For release notes: