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

RTL issues with multiCopter #8389

Closed
santiago3dr opened this issue Nov 29, 2017 · 10 comments
Closed

RTL issues with multiCopter #8389

santiago3dr opened this issue Nov 29, 2017 · 10 comments
Assignees
Labels

Comments

@santiago3dr
Copy link

If engaging RTL multiple times in the same flight, the second+ instance of RTL goes towards a previously set path by the first RTL engagement.
Couple examples:
https://review.px4.io/plot_app?log=ed176590-4352-4516-b5ed-3892436d6477
https://review.px4.io/plot_app?log=d9c63fd1-1224-4be7-8985-932a2f670c78

@dagar

@LorenzMeier
Copy link
Member

I think we need to fix this pre-release

@dagar dagar added the bug label Nov 29, 2017
@sanderux
Copy link
Member

I cant reproduce this in SITL, also i don't see this behavior in the logs posted but they are a bit hard to read.

@LorenzMeier
Copy link
Member

@Stifael Could you have a look?

@Stifael
Copy link
Contributor

Stifael commented Dec 4, 2017

I am also not able to reproduce it. so far I looked at the first log, and it seems that navigator sends out the correct setpoints.

@LorenzMeier
Copy link
Member

@santiago3dr Could you please try to reproduce this?

@santiago3dr
Copy link
Author

reproduced with pixracer (V4) running current master b21af47
https://logs.px4.io/plot_app?log=0a53df91-c2fe-4040-b0c1-451cacef2d09

of the 3 instances rtl was engaged, 1 and 2 took direct path home while the third attempt headed towards a path previously set by a prior instance of rtl
rtl

@Stifael
Copy link
Contributor

Stifael commented Dec 5, 2017

@santiago3dr thanks. it is a much simpler log and I think I just found a pattern

@Stifael
Copy link
Contributor

Stifael commented Dec 5, 2017

I can reproduce it now

@Stifael
Copy link
Contributor

Stifael commented Dec 5, 2017

here is a PR which solves the issue.
#8412

@dagar
Copy link
Member

dagar commented Dec 8, 2017

Fixed! Thanks Dennis.

@dagar dagar closed this as completed Dec 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants