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

Visualize101.ino: Simplify sensor reading #16

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

Conversation

eriknyquist
Copy link

@eriknyquist eriknyquist commented Oct 11, 2016

CurieIMU now has dataReady() functions to check for newly available
accel/gyro data, and also readAccelerometer/GyroScaled(), which reads
then sensor and returns values scaled according to the configured range.
Using these functions in the Visualize101 example makes the source much
simpler

@eriknyquist
Copy link
Author

arduino/ArduinoCore-arc32#317 has been merged, this can also be merged now.

@PaulStoffregen
Copy link
Contributor

PaulStoffregen commented Oct 15, 2016

Has this new 101 core been released to the boards manager downloads?

If it's only been merged on github, but not yet officially released via the boards manger, then updating this library could break things for 101 users.

With this in mind, just tell me when you want it merged. They're your users....

@eriknyquist
Copy link
Author

Good point @PaulStoffregen-- I was not thinking straight on Friday. So don't merge yet- I'll let you know when our next release is available from the IDE library manager (currently it is not).

@eriknyquist eriknyquist changed the title Visualize101.ino: use CurieIMU.*DataReady() methods Visualize101.ino: Simplify sensor reading Jan 24, 2017
@eriknyquist
Copy link
Author

Updated to use readAccelerometerScaled, and readGyroScaled, which greatly simplifies the example (manual conversion no longer required).

CurieIMU now has dataReady() functions to check for newly available
accel/gyro data, and also readAccelerometer/GyroScaled(), which reads
then sensor and returns values scaled according to the configured range.
Using these functions in the Visualize101 example makes the source much
simpler
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

None yet

3 participants