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

Thrust value oscillating (repeatedly dropping to 0) #13507

Closed
tomasz-lewicki opened this issue Nov 18, 2019 · 8 comments
Closed

Thrust value oscillating (repeatedly dropping to 0) #13507

tomasz-lewicki opened this issue Nov 18, 2019 · 8 comments

Comments

@tomasz-lewicki
Copy link

Hey,

My Pixhawk 4 mini hexarotor with PX4 1.9.2 release wasn't maintaining altitude while hovering, so I increased the mid throttle from 50% to 60%.

What I got is this super odd thrust oscillation, while hovering (the thrust value is going to 0 for a short period of time -> see the logs). Even though I'm trying to land, it's only reacting to increasing the throttle, but not to not lowering -> also visible in the logs. I had to use kill switch.

The oscillations occur across all modes.
After going back to 50% mid-throttle the problem disappears.

The battery warnings are result of the voltage sag (the actual voltage was 3.75V/cell and the default BAT_V_EMPTY is pretty high, at 3.50V/cell).

Logs:
https://logs.px4.io/plot_app?log=ececa147-2ad7-4aa7-9e0a-bcc0761c06af

@tomasz-lewicki
Copy link
Author

@julianoes It really seems that these really large changes in thrust are not correlated to with any sensor input or state estimates. Could you advise how I could get started on investigating/fixing this?

Could it be a repeatedly occurring exception in code that causes the thrust to reset to 0?

@steffensol
Copy link

This seems like it is the land detector acting up again. Multiple issues have been flagged for this, and it seems that the detector is VERY sensitive to hover throttle and minimum throttle values.

I've used "flightplot" to plot out the ground detection algorithm, see the attached plot showing the detection occurring.
ground-contact

This problem has been seen across multiple vehicles, vehicle types, and has been flagged as issues here, with different names and explanations - but they all seem to be linked in my opinion.

#13124
#11683
#13231

I'd say based on your logs that you probably don't need the 60% thrust to hover.
It could very well be the PID values of the position-controller acting up because it seems like it is not following the Z-setpoint at all.

PID-Z-VEL

@RomanBapst
Copy link
Contributor

Multiple issues have been flagged for this, and it seems that the detector is VERY sensitive to hover throttle and minimum throttle values.

@steffensol Yes, and that's why @bresch is working on the hover thrust estimator.
#13981

@steffensol
Copy link

That's brilliant!
However, it still doesn't solve the case that the landing detector needs more love&care. In our case, we have the hover thrust set up perfectly, within 1%. Our minimum thrust, however, is only 15% lower than the hover thrust - which causes a "low thrust situation" to trigger very quickly.

@RomanBapst
Copy link
Contributor

RomanBapst commented Feb 17, 2020

@steffensol What's the reason for it being only 15% lower. Is it a tailsitter :-)
I've personally created a PR once which does not allow detecting ground contact only by a low thrust situation but people had some concerns. I've seen vehicles crash due to this effect so I do think it's quite serious.

@steffensol
Copy link

The main reason is that we have very large propellers running low RPM, to give high efficiency (We're getting flight times in excess of 1h). Unfortunately this also sometimes leads to loss-of-sync with the ESCs at low throttle. Hover is currently at 40%, and the minimum at 25% - if we go any lower, flight performance is very stuttery.

@stale
Copy link

stale bot commented May 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@dagar
Copy link
Member

dagar commented Aug 2, 2020

MC land detector ground_contact detection fixes have been merged in #15083.

@dagar dagar closed this as completed Aug 2, 2020
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

5 participants