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

Accelerometer axes #398

Closed
jps2000 opened this issue Nov 7, 2018 · 8 comments
Closed

Accelerometer axes #398

jps2000 opened this issue Nov 7, 2018 · 8 comments

Comments

@jps2000
Copy link

jps2000 commented Nov 7, 2018

Problem

Accelerometer report x, y, z axes. Rotating the Ganglion revealed that the sign of axes are not right hand cartesian coordinates ( Turning +x into +y axis should point in +z direction (right hand screw)

Expected

Right hand coordinate repersentation
Hence z should change sign or x exchanged with y

Operating System and Version

Windows 7 professional 64bit

GUI Version

4.0.2 ( probably in all versions)

Type of OpenBCI Board

Ganglion

Are you using a WiFi Shield?

No

@jps2000 jps2000 changed the title Accelerometer display in playback in 4.0.2 Accelerometer axes Nov 8, 2018
@jps2000
Copy link
Author

jps2000 commented Nov 12, 2018

Note that acc chip has changed pcb side between cyton and ganglion. ( hence x and z axis changed sign, y remains)
Preferably ganglion should deliver same axes as cyton. This should be taken into account in ganglion stack not in GUI as it has to do with hardware.

@retiutut
Copy link
Member

Note that acc chip has changed pcb side between cyton and ganglion. 👍

The following pictures document this issue. Cyton and Ganglion were placed side by side using GUI on two machines, then rotated in the same way. The GUI shows different results:

  • Flat and facing up
    openbci-2018-11-20_07-37-53 - cyton 1
    openbci-2018-11-20_07-37-48 - ganglion - 1

  • Rotation 1
    openbci-2018-11-20_07-38-24 - cyton 2
    openbci-2018-11-20_07-38-19 - ganglion - 2

  • Rotation 2
    openbci-2018-11-20_07-38-51 - cyton 3
    openbci-2018-11-20_07-39-04 - ganglion - 3

@jps2000
Copy link
Author

jps2000 commented Nov 20, 2018

Yeah thats what I discovered.
Assuming Cyton as the reference, Ganglion should deliver same acc output (preferably).

@retiutut
Copy link
Member

retiutut commented Nov 20, 2018

@andrewjaykeller Can I go ahead and fix this in the Accelerometer widget using Cyton as a reference?

@andrewjaykeller
Copy link

andrewjaykeller commented Nov 20, 2018

So X and Z are multipled by -1?

@retiutut
Copy link
Member

So X and Z are multiples by -1?

Z can be inverted for Ganglion.

X and Y may need to be swapped?

@jps2000
Copy link
Author

jps2000 commented Nov 21, 2018

Considering Cyton as reference:
In displaying Ganglion , Z need to be inverted AND X and Y need to be swapped but do not change sign.

@jps2000
Copy link
Author

jps2000 commented Feb 11, 2019

Solved in 4.1.0
Thank you

@jps2000 jps2000 closed this as completed Feb 11, 2019
OpenBCI SW Dev automation moved this from To do to Done Feb 11, 2019
@daniellasry daniellasry mentioned this issue Mar 9, 2019
retiutut added a commit to retiutut/OpenBCI_GUI that referenced this issue Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

No branches or pull requests

3 participants