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

Update IMU_Zero.ino #280

Merged
merged 1 commit into from
Dec 5, 2016
Merged

Update IMU_Zero.ino #280

merged 1 commit into from
Dec 5, 2016

Conversation

RobertRFenichel
Copy link
Contributor

Added delay to keep from reading IMU faster than 200 Hz. Without the delays, program hangs on some devices. Comments expanded.

  Added delay to keep from reading IMU faster than 200 Hz.  Without the delays, program hangs on some devices.  Comments expanded.
@jrowberg jrowberg merged commit 2f29979 into jrowberg:master Dec 5, 2016
@piroozn
Copy link

piroozn commented Dec 5, 2016

@RobertRFenichel
Please do not add delay. We have to make these libraries more robust and correct by using status bits for the interface that indicate the status of the Fifo and if we need to wait or just read it.
Delay is a bad idea, Please don't use delay.

@PaulStoffregen
Copy link
Collaborator

Especially for gyroscope data, reading at a consistent known rate which corresponds to the actual sensor acuisition rate is essential. Converting angular rates to position involves integrating over time. Without consistent acquisition rate, you can end up with an impressively fast scrolling list of numbers that are much less useful for filters or fusion algorithms to actually do useful for computing orientation.

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.

4 participants