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

Add function to enable 16G readings from accelerometer #107

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

BlaiseSaunders
Copy link

@BlaiseSaunders BlaiseSaunders commented Jul 23, 2021

Describe the scope of your change--i.e. what the change does and what parts of the code were modified.

This change adds a function that enables the accelerometer on the BNO to read up to 16G instead of the default 8G, which was needed by our team for measuring rocket acceleration as it was capping out on previous launches. I've done my best to try and bring the changes in line with the style that I've seen in this repo

I made some small changes to the main file and header to add in the function and required variables

Describe any known limitations with your change.

This change only works with the BNO055 out of fusion mode, more a limitation of the device

Please run any tests or examples that can exercise your modified code.

This modified driver was used in a rocket launch and successfully measured up to 16G.

If there's any further changes you'd like me to make or anything I can add to get this up to scratch to be merged, please let me know and I will action as soon as I can. Thanks :)

@ladyada
Copy link
Member

ladyada commented Jul 24, 2021

hi, as-is, this is a single funciton that can't be undone, it would be best if it were refactored to allow setting any range (including 16G)

@BlaiseSaunders
Copy link
Author

hi, as-is, this is a single funciton that can't be undone, it would be best if it were refactored to allow setting any range (including 16G)

Hi Ldyada, thanks for your reply! I'll have a crack getting this written up and report back :)

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.

2 participants