Skip to content

gyro values return in degrees per second but CP standard is radians per second #9

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

Closed
kevinjwalters opened this issue Mar 7, 2020 · 4 comments
Assignees

Comments

@kevinjwalters
Copy link

kevinjwalters commented Mar 7, 2020

Based on a quick test with an Adafruit CLUE using the https://github.com/adafruit/Adafruit_CircuitPython_CLUE library which is passthru for gyro value, the gyro return appears to be in degrees per second. The code in the example (and README) also suggest this is the case in the textual description of units: https://github.com/adafruit/Adafruit_CircuitPython_LSM6DS/blob/master/examples/lsm6ds33_simpletest.py#L12

https://circuitpython.readthedocs.io/en/latest/docs/design_guide.html#sensor-properties-and-units specifies gyro as x, y, z radians per second (NB: inconsistent with orientation specification; CP is consistent with C/Arduino spec here for these two values, see https://learn.adafruit.com/using-the-adafruit-unified-sensor-driver/how-does-it-work#standardised-si-units-for-sensor-data-2-2)

Worth independent verification via datasheet or empircally before changing anything here...

@ladyada
Copy link
Member

ladyada commented Mar 7, 2020

could be! @siddacious ya can take a look next week

@kevinjwalters
Copy link
Author

BTW, you can dangle a board by its USB lead and twist that to get approximately 60 rpm as a crude measurement test if you don't have a spinnable platform to hand.

@kevinjwalters
Copy link
Author

Similar issue over on adafruit/Adafruit_BNO055#50 but perhaps that can't be changed because too much code + users would be affected.

@siddacious
Copy link
Contributor

sorry for the ate reply but yes, I can confirm that we're reporting dps not rads. I'll put in a fix and do a major release since the API is changing despite it also being a fix

@jepler jepler closed this as completed in 350d313 Apr 20, 2020
jepler added a commit that referenced this issue Apr 20, 2020
change from dps->rads for gyro to fix #9
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

No branches or pull requests

3 participants