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

Offboard mode can lead to crash #84

Closed
julianoes opened this issue Sep 21, 2017 · 6 comments
Closed

Offboard mode can lead to crash #84

julianoes opened this issue Sep 21, 2017 · 6 comments

Comments

@julianoes
Copy link
Collaborator

The offboard functionality needs some testing and bugfixing because it has lead to crashes in the past.

It seems to work fine when you start in NED mode first but can lead to a crash if started in body mode.

@shakthi-prashanth-m
Copy link
Contributor

@julianoes : I didn't observe crash in this case.
I created a sample app to test offboard mode (using the code from integration_tests) with both NED & BODY mode.
Neither of them crashed. Any idea what caused crash ?

@shakthi-prashanth-m
Copy link
Contributor

$ DroneCore/example/offboard_velocity/build$ ./offboard NED
[06:46:44|Info ] New device on: 127.0.0.1:14557 (udp_connection.cpp:212)
In Air
sent velocity ned once
Offboard started
Let yaw settle...
sent velocity ned 628 times
sent velocity ned 400 times
sent velocity ned 400 times
sent velocity ned 400 times
sent velocity ned 400 times
Offboard stopped
Landed
Disarm failed:Command denied, not landed
-------------------------------------------------------------------------------
$ DroneCore/example/offboard_velocity/build$ ./offboard BODY
[06:47:34|Info ] New device on: 127.0.0.1:14557 (udp_connection.cpp:212)
In Air
sent velocity ned once
Offboard started
Turn around yaw & climb
Turn back
Wait for a bit
Fly a circle
Wait for a bit
Fly a circle sideways...
Wait for a bit
Offboard stopped
Landed
Disarmed

@shakthi-prashanth-m shakthi-prashanth-m self-assigned this Oct 11, 2017
@julianoes
Copy link
Collaborator Author

@shakthi-prashanth-m thank you. Could you make a pull request and add this test either as an example and/or as an integration test?

I will run this also against an internal version of Firmware to see if that's where the bug comes from.

@shakthi-prashanth-m
Copy link
Contributor

Sure @julianoes. I added it as an example. Please check.

@julianoes
Copy link
Collaborator Author

This was a bug in our internal version of Firmware.

It was fixed through:
PX4/PX4-Autopilot@c41185a
PX4/PX4-Autopilot@d9cec60

@shakthi-prashanth-m
Copy link
Contributor

Cool 😊 Thanks @julianoes

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

No branches or pull requests

2 participants