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

MPU-6050 No-DMP mode #291

Closed
wants to merge 14 commits into from
Closed

Conversation

furrycoding
Copy link

Adds a new type of sensor that uses MPU-6050, but without the DMP(sensor fusion in software)

  • Pros:
    • No need to keep MPUs sill/flat on the ground when powering on
  • Cons:
    • Potentially higher drift
    • Manual calibration is basically required

Uses the same sensor fusion as BMI160

@furrycoding
Copy link
Author

TODO: Verify that rest detection for TCAL actually works(MPUs accel/gyro tend to be quite noisy)

I need to find clean way to modify rest detection config however

@furrycoding
Copy link
Author

Tested tracker with the last commit for a few hours - TCAL seems to work!

Now to figure out how to deal with conflicts lol

@furrycoding
Copy link
Author

Rebased the branch to 0.4.0
Doing some testing right now, but eveything seems to be working well

@furrycoding furrycoding marked this pull request as ready for review December 16, 2023 05:54
@furrycoding
Copy link
Author

Having some weird issues with TCAL
I think on some MPUs the accel noise might increase with temperature
Meaning past some temperature it could stop detecting rest
Need to fine-tune the accel threshold

src/consts.h Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@wb1016
Copy link

wb1016 commented Feb 8, 2024

works on ESP8266 and ESP32C3. one thing i would suggest to change is display name of IMU.
asdfasdfsfadsdafasdcasdf

@furrycoding
Copy link
Author

one thing i would suggest to change is display name of IMU. asdfasdfsfadsdafasdcasdf

Would need to add a new IMU type in the server, SolarXR, and finally in the UI. I do agree that it'd be nice to know what mode the MPU is in, but using the IMU type for this would require too many changes

@furrycoding
Copy link
Author

Rebased onto latest main

furrycoding and others added 2 commits March 15, 2024 05:09
Butterscotch can english better than me

Co-authored-by: Butterscotch! <[email protected]>
@unlogisch04
Copy link
Contributor

I'm not sure if there is still need for this PR, as the softfusion branch has a MPU6050 with softfusion available.

@furrycoding
Copy link
Author

This branch still has temperature calibration, which sfusion currently lacks. However, that'll soon likely change. So there's no point in this PR anymore. (However the branch itself has some use)

@furrycoding furrycoding closed this Jul 4, 2024
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

Successfully merging this pull request may close these issues.

4 participants