-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Commander check reporting improvements #22886
Conversation
Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
Such that the focus is on the important keywords. Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
@@ -402,15 +402,15 @@ | |||
}, | |||
"2": { | |||
"name": "fallback_posctrl", | |||
"description": "fallback to Position mode" | |||
"description": "Position mode" |
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.
Fallback isn't important context?
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.
I don't think so. All the fields in this enum are fallbacks in the end.
See my screen recording above to see how it looks for the operator. I think in the context it's enough.
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.
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.
Solved Problem
The failsafe event messages are a bit long.
Solution
Make them shorter.
That's how it would look like now:
https://github.com/PX4/PX4-Autopilot/assets/26798987/ece1ea4e-74de-473c-8458-a7c24d986d6e
Changelog Entry
For release notes: