search

LiquidCGS / FastIMU

the IMU library to rule them all (wip)
MIT License
122 stars 15 forks source link

Fixed divide by zero error when calibrating MPU-6886 #25

Closed floBik closed 1 week ago

floBik commented 1 week ago

Fixed the crash due to division by zero when calibrating the MPU6886 mentioned in https://github.com/LiquidCGS/FastIMU/issues/21. The problem was most likely caused by disabling the FIFO and reading the FIFO_COUNT register in the wrong order.

floBik commented 1 week ago

Hey @LiquidCGS it would be great if you had some time to take a look at this PR. There is a project of mine that depends on it.

LiquidCGS commented 1 week ago

Hey @LiquidCGS it would be great if you had some time to take a look at this PR. There is a project of mine that depends on it.

On it!

LiquidCGS commented 1 week ago

I don't have an MPU6886 to test at the moment, it's odd that this fixes it since almost that exact same code works for all the other IMU's of the same family, but hey, if it works then that's what matters.

floBik commented 1 week ago

Thanks! Yes its indeed very confusing...