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

FreeImuPlugin.cs modified for yaw,pitch,roll,x,y,z #50

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

Conversation

SimulateIT
Copy link

Modified FreeImuPlugin

Added

data.X = ParseFloat(values[3]);
data.Y = ParseFloat(values[4]);
data.Z = ParseFloat(values[5]);

Also modified ComDevicePlugin.cs

Works only, if all 6 axis (yaw,pitch,roll,x,y,z) send via serial.

Have to add DOF-Dropdown in plugin settings, to select "yaw,pitch,roll" or "yaw,pitch,roll,x,y,z"

Modified FreeImuPlugin

Added

data.X = ParseFloat(values[3]);
data.Y = ParseFloat(values[4]);
data.Z = ParseFloat(values[5]);

Perhaps someone could compile this, that i could test with my Headtracking-Helmet.
@cyberluke
Copy link

I am hardware developer and have Bosch BNO055 custom wireless bracelet with this IMU.

I send packets of yaw,pitch,roll in quaternion or classic values from Gyroscope.

Accelerometer data should be optional, not required and relative IMU device (without IR sensors) should be supported.

You should not break compatibility and require 6 axis.

@cyberluke
Copy link

What you presented is merely an ugly hack. You need to make it configurable or develop your own custom device plugin.

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.

2 participants