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

Kill switch now also works for fbw mode #1951

Merged
merged 1 commit into from
Nov 16, 2016

Conversation

kevindehecker
Copy link
Contributor

I had a few close calls with the kill switch not working (with my Iris+ with Pixhawk and the 3dr frysky remote ) when it is in fbw mode, so...

Tested and works with the Iris+

@podhrmic
Copy link
Member

@kevindehecker What do you mean by fbw mode? I noticed that no matter what my radio settings are, the MODE0 is always RC_DIRECT, but MODE1 and MODE2 can be changed to whatever I need. Does this take care of that too?

@kevindehecker
Copy link
Contributor Author

No, unfortunately not.

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. I agree we should add one.

@podhrmic
Copy link
Member

OK, merging as is and hoping for a way to disable the RC direct in the future PR.

@podhrmic podhrmic merged commit c7f0c04 into paparazzi:master Nov 16, 2016
@kevindehecker kevindehecker deleted the fbw_kill 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.

2 participants