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

Occasional erroneous values from i2c magnetometer #5504

Closed
jschall opened this issue Jan 5, 2017 · 6 comments
Closed

Occasional erroneous values from i2c magnetometer #5504

jschall opened this issue Jan 5, 2017 · 6 comments
Labels

Comments

@jschall
Copy link
Contributor

jschall commented Jan 5, 2017

Issue details

This is a plot over 15 hours:
magnetometerbadsamples

There are occasionally these huge outliers from the first magnetometer.

Version

Based on dab0d8e

Platform

[ ] All
[ ] AntennaTracker
[x] Copter
[ ] Plane
[ ] Rover

Airframe type

Quad

Hardware type

Pixhawk 2

@gmorph
Copy link
Contributor

gmorph commented Jan 5, 2017

Did you notice anything else happen in the dataflash logs at the time of the blips?
Thanks, Grant.

@jschall
Copy link
Contributor Author

jschall commented Jan 6, 2017

Nothing at the same time.

@OXINARF OXINARF added the Library label Jan 6, 2017
@tridge
Copy link
Contributor

tridge commented Jan 11, 2017

i2c doesn't have any checksums, so this could just be bus noise.
We could try and include some sort of spike filter, but really the EKF is already doing that for us.

@tridge
Copy link
Contributor

tridge commented Jan 11, 2017

if we double sampled we may be able to detect these, but it would halve the sample rate and that could cause aliasing issues

@OXINARF
Copy link
Member

OXINARF commented Jun 2, 2017

Can this be closed?

@WickedShell
Copy link
Contributor

I'm closing it and pointing to the I2C DNF as the fix. I could reproduce this problem before with noise on the I2C bus, #5935 if anyone disagrees this can be reopened, but at a minimum would need to prove that it still happens now after that patch.

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