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

Added define to disable rc-direct fbw mode manual #1953

Merged

Conversation

kevindehecker
Copy link
Contributor

FBW mode (fly by wire) is a feature implemented on the Pixhawk, which (e.g. in case the AP crashes) enables the pilot to always override with RC control. This mode makes sense in case of e.g. fixed wing operation, which often don't necessarily require stabilization to be human-controllable. However, for a quadrotor such as the Iris+ it's useless, unless an IMU is added to the FBW processor such as in the Outback board. (and although the Pixhawk has two IMU's, both are connected to the AP)

The Pixhawk has an auxiliary processor (its got an STM32F1 FBW processor and an STM32F4 AP processor) to take care of the servo commands and RC receiver, and a main processor which runs the AP and stuff. The RC commands are passed on to the AP by the FBW. And before that happens, the FBW interprets the RC commands in order to not depend on the AP. So, if the RC command says FBW, the AP cannot override.

Currently there is no define to disable RC-Direct ( -->"FBW mode") from an airframe xml. This PR adds one.

@gautierhattenberger
Copy link
Member

Looks okay, except I don't really like the name FBW_ENABLE_MANUAL. I can't really find a really better name. Maybe FBW_MODE_AUTO_ONLY ?
You should add the documentation with this new setting in the code anyway.

@kevindehecker
Copy link
Contributor Author

Done

@gautierhattenberger gautierhattenberger merged commit b9b5a70 into paparazzi:master Nov 21, 2016
@kevindehecker kevindehecker deleted the add_fbwmode_define branch February 2, 2017 17:23
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.

None yet

2 participants