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 wom service and modify gyro data conversion #72

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Lahorde
Copy link

@Lahorde Lahorde commented Mar 30, 2016

No description provided.

Lahorde added 2 commits March 30, 2016 11:17

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
var xG = data.readInt16LE(0) / 128.0;
var yG = data.readInt16LE(2) / 128.0;
var zG = data.readInt16LE(4) / 128.0;
// rotation, unit deg/s, range -250, +250
Copy link
Owner

Choose a reason for hiding this comment

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

This change looks good.

@Lahorde
Copy link
Author

Lahorde commented Apr 4, 2016

WOM is based on accelerometer when the high passed sample from any axis has an absolute value exceeds a user-programmable threshold.
Consequently, for those sensing magnetic field without moving sensortag, wom will be a disadvantage!

I've tested #64 issue with a simple test code and I got magneto values :
https://gist.github.com/Lahorde/9628db14b7319bb210d9b3c6d4f8cbc8

Is issue still reproducible with a recent sensortag firmware?

@sandeepmistry
Copy link
Owner

Is issue still reproducible with a recent sensortag firmware?

@Lahorde for me yes. @zywind any comments on this?

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

2 participants