Skip to content

modified example in Arduino style with accompanying Processing sketch #76

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
wants to merge 1 commit into from

Conversation

hbisby
Copy link
Contributor

@hbisby hbisby commented Dec 2, 2015

The RawIMUSerialData example has been modified to the code of a tutorial written which uses the IMU data to calculate Euler angles and present the orientation of the board in Processing (sketch is commented at the bottom of RawImuSerialData.ino). Ifdefs have been removed (replaced by if statement for calibration) as well as the option for binary output, since the format was considered too complicated for Arduino style. Types have also been changed to Arduino, e.g uint16_t has been changed to int.

@sandeepmistry
Copy link
Contributor

@facchinm @tigoe do we want this change to be included? If so, I can add rebasing it and updating to my list of things to look at.

@tigoe
Copy link
Member

tigoe commented Feb 10, 2016

I'd rather see the example updated to the new IMU API first.

@kitsunami
Copy link

@tigoe, is this pull request still valid?

@kitsunami kitsunami added this to the Bellatrix milestone Feb 24, 2016
@tigoe
Copy link
Member

tigoe commented Feb 25, 2016

Probably, but I’d defer to Sandeep on it. Helena made these changes first, then Sandeep made bigger changes. I assume he had incorporated all of her changes, but he could say for sure. On the road at the moment, and not able to look at the full diff. @sandeepmistry, comments?
Tom

On Feb 24, 2016, at 2:55 PM, kitsunami notifications@github.com wrote:

@tigoe https://github.com/tigoe, is this pull request still valid?


Reply to this email directly or view it on GitHub #76 (comment).

@sandeepmistry
Copy link
Contributor

@kitsunami please close this, instead of bundling an example that depends on a the external MadgwickAHRS library, we will make this into a tutorial.

@tigoe the MadgwickAHRS library is used to calculate the yaw, roll and pitch from the raw accelerometer and gyroscope values. So, it won't work out of the box unless the user adds the library on their system.

cc/ @facchinm @agdl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants