Skip to content
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

Copter: Notify that the radio failsafe is disabled #19721

Conversation

muramura
Copy link
Contributor

@muramura muramura commented Jan 7, 2022

Disables the radio fail-safe during automatic flight.
This vehicle can be operated with a radio.
The operator does not get passively that the radio failsafe is disabled.
I learned that when the radio receiver of the radio is abnormal, it notifies the active PWM.
When I manually operate the vehicle, it notifies me that the radio failsafe is disabled.
I notify the message to the GCS, which in turn notifies the operator by voice and displays the message.

https://ardupilot.org/copter/docs/radio-failsafe.html#receiver-configuration-for-low-throttle-method

AFTER
Screenshot from 2022-01-08 06-52-31

ArduCopter/AP_Arming.cpp Outdated Show resolved Hide resolved
@muramura muramura force-pushed the AP_Notify_that_the_radio_failsafe_is_disabled branch from 29f62b8 to 72b2d10 Compare January 7, 2022 23:35
@muramura
Copy link
Contributor Author

muramura commented Jan 7, 2022

@nkruzan san
Thanks for the comment.
I agreed.

Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are vast numbers of parameters that the user might not set up.

In this case they might not actually be using RC on their vehicle, so don't need the failsafe.

This is something where @amilcarlucas 's tool is the correct thing to do. We can't afford to check every parameter for every mistake; too much flash.

Sorry, closing this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants