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

Altitude hold not working #170

Closed
ataffanel opened this issue Nov 28, 2016 · 4 comments
Closed

Altitude hold not working #170

ataffanel opened this issue Nov 28, 2016 · 4 comments
Milestone

Comments

@ataffanel
Copy link
Member

In the current master branch, altitude hold seems to be broken. This seems to come from the work on the position controller (see #136) that broke the control modes contained in the setpoint structure.

@stephanbro
Copy link
Contributor

This has definitely been broken as part of the position controller changes, as you suspect. I have a fix I will try to push today.

@ataffanel
Copy link
Member Author

I was about to look at it but if you already have a fix that is great! thanks.

@ataffanel
Copy link
Member Author

With PR #172 the altitude hold now holds a constant vertical velocity. The current altitude estimator is not outputting vertical velocity which would explain this phenomena.

@ataffanel
Copy link
Member Author

39831c3 fixes altitude-hold. I also tried to tune a bit the Z estimator (there was an oscillation between the thrust and the height measurement, fixed by filtering the pressure sensor a bit harder).

@ataffanel ataffanel added this to the Next version milestone Nov 28, 2016
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

No branches or pull requests

2 participants