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

Improvements on datarate, added dataReady function, fixed accelerometer and gyroscope divisor #20

Merged
merged 6 commits into from
Aug 26, 2024

Conversation

uutzinger
Copy link
Contributor

Removed sleep() in functions reading accelerometer, gyroscope, magnetometer
Added dataReady function
Fixed accelerometer and gyroscope divisor
Made example for measuring data rate
Updated data_rate settings in examples
Added ability set gravity constant

…etter

Providing data ready function
Allow to set gravity constant
Removed sleep statement when reading accelerometer, gyroscope and magnetometer
Repaired accelerometer_data_rate and gyroscope_data_rate function
Updated examples that use data_rate setter
Created example measuring data_rate
Fixed gravity setter
Added i2c address to 20649 examples
Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @uutzinger!

The actions task failed due to code format. If you set up and run pre-commit and then commit the resulting changes to this branch it should help the actions pass that step.

I think don't think we want the build/lib/ directory or the file inside of it to be checked in to the repo. Can you remove that one as well please.

Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

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

This looks good to me. I tested the existing simpletest and full test scripts on a feather S3 TFT with https://www.adafruit.com/product/4554

I also tested the new example scripts that make use of the data_ready functionality and confirmed the resolution of the outputs changing in Mu serial plotter.

I pushed some commits to merge main, resolve conflicts, and to rename the property to snake case data_ready to be consistent the rest of this library and our others.

Thanks for adding this functionality @uutzinger

@FoamyGuy FoamyGuy merged commit 0a76595 into adafruit:main Aug 26, 2024
1 check passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Aug 27, 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.

2 participants