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

[sfusion, BMI270] Perform gryo motionless calibration before gyro offset calibration #367

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

jabberrock
Copy link
Contributor

@jabberrock jabberrock commented Nov 23, 2024

On a normal reset, we set the CRT (motionless) registers at startup, and then apply the calibrated gryo offset to every gyro sample.

Previously, we performed gyro offset calibration before CRT calibration. This means that we are actually calculating the gyro offset based on a CRT of 0, instead of the actual CRT.

This change performs CRT calibration before gyro calibration, so that the gryo offset is based on the actual CRT.

Cherry-picked from:
l0ud@2424b2f
l0ud@cf34988

…set calibration

On a normal reset, we set the CRT (motionless) registers at startup, and then apply the calibrated gryo offset to every gyro sample.

Previously, we performed gyro offset calibration before CRT calibration. This means that we are actually calculating the gyro offset based on a CRT of 0, instead of the actual CRT.

This change performs CRT calibration before gyro calibration, so that the gryo offset is based on the actual CRT.
Copy link
Contributor

@l0ud l0ud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ButterscotchV ButterscotchV merged commit 0ad955d into SlimeVR:main Nov 25, 2024
2 checks passed
@jabberrock jabberrock deleted the jabber-crt-before-gyro branch November 25, 2024 01:47
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.

3 participants