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

Copter: prevent bad Z-vibrations causing rapid climb. #1009

Closed
AndKe opened this issue Apr 21, 2014 · 6 comments
Closed

Copter: prevent bad Z-vibrations causing rapid climb. #1009

AndKe opened this issue Apr 21, 2014 · 6 comments

Comments

@AndKe
Copy link
Contributor

AndKe commented Apr 21, 2014

Here's latest victim of AccZ vibrations, http://diydrones.com/forum/topics/fly-away-please-help-me-figure-out-why

please note that IMU2 provided good data.
I know that this is a user/assembly error, but still, there should be some handling of this scenario - after all, we can still detect an altitude gain that contradicts desired altitude/(input)

@rmackay9 rmackay9 added this to the AC 3.2.0 milestone Apr 21, 2014
@rmackay9
Copy link
Contributor

AC3.2 will include mixing of the two accelerometers on the Pixhawk (AC3.1.3 only uses the MPU6k). We believe that luckily because one accelerometer runs at 800hz and the other at 1000hz that aliasing will be much less likely.

It has not been thoroughly tested yet but in one test we saw one accelerometer which freaked out due to aliasing but the other did not.

@rmackay9 rmackay9 changed the title Prevent Z-vibrations causing rapid climb. Copter: prevent bad Z-vibrations causing rapid climb. Apr 21, 2014
@rmackay9
Copy link
Contributor

While testing in the simulator I found that the EKF (fast CPU boards only like Pixhawk) does not have this problem. With a bias of 2m/s/s the vehicle only climbed 2m. On the APM2 the vehicle climbed 82m.

@rmackay9 rmackay9 modified the milestones: AC 3.3.0, AC 3.2.0 Nov 6, 2014
@R-Lefebvre
Copy link
Contributor

I wonder if this can be closed now? Better filtering of the accels. EKF. Bad sensor rejection... all coming with 3.3.

@rmackay9
Copy link
Contributor

Yes, the increase in the accel range from 8G to 16G should fix this. We've never actually tested this in a copter but I think we are fairly confident.

@vmayoral
Copy link
Contributor

After these changes we actually started seeing the copters climbing. Refer to #2026

@rmackay9
Copy link
Contributor

I'm closing this issue after all the work we've done to improve the vibration handling. We can open a new issue when/if we see it happen with AC3.3 (or higher) versions.

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

4 participants