Skip to content

Example 20 Bitwise OR Fix #23

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rcxking
Copy link

@rcxking rcxking commented Jul 31, 2024

This commit fixes a bug in the Example 20 - Calibration Settings where logical OR was used instead of bitwise OR. The result is that the IMU's setCalibrationConfig() was evaluated with value 1 rather than 7.

This commit fixes a bug in the Example 20 - Calibration Settings where
logical OR was used instead of bitwise OR.  The result is that the IMU's
`setCalibrationConfig()` was evaluated with value 1 rather than 7.

Signed-off-by: Bryant Pong <[email protected]>
@rcxking
Copy link
Author

rcxking commented Jul 31, 2024

Hello, I'm using the BNO086 Sparkfun board for a robotics competition and happened to see this bug when I was looking at examples on how to calibrate the sensor. As this is my first PR to this repo I read the contributing guidelines but got a 404 error when trying to find the requested release-candidate branch; please let me know if I should not PR to main. Thank you!

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.

1 participant