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 support for FIFO for accelerometer and gyroscope data #21

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

Conversation

finnboeger
Copy link

This adds support for using the FIFO queue to store the values measured by the accelerometer and the gyroscope and to read back that data quickly.

It is based on the code from https://github.com/wollewald/ICM20948_WE but includes a small improvement to extract the data faster.

Using the FIFO it is possible to sample data continuously at the maximum sample rate without dropping a single measurement by

  1. stopping the sampling
  2. getting the number of entries in the queue
  3. restarting the sampling
  4. reading from the queue.

When sampling both the accelerometer and the gyroscope, the order is always accelerometer data first, then gyroscope data, assuming the same sample rate. Using this code with a different sample rate for the accelerometer and gyroscope is untested behavior.

@Gadgetoid Gadgetoid added the enhancement New feature or request label Jan 20, 2023
@Gadgetoid
Copy link
Member

Sorry for the radio silence- this looks interesting, thank you!

I don't know when we'll get around to reviewing & merging, but we're starting to build a bit more process around that.

In the interim, if anyone's driving by here and gives this code a try then feedback would be appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants