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

Introduced IMU 2.6 #594

Merged
merged 2 commits into from
Aug 18, 2019
Merged

Introduced IMU 2.6 #594

merged 2 commits into from
Aug 18, 2019

Conversation

pattacini
Copy link
Member

@pattacini pattacini commented Aug 12, 2019

As per request #570, this PR introduces IMU version 2.6.
Changes are regarded with:

  • iKin to declare the new kinematics.
    Differently from what I planned, HN matrices are accumulated to adapt to the current code.
  • iKinGazeCtrl to extend the set of allowed IMU's.

@Nicogene
Copy link
Member

I tried it on iCubGenova02 using iKinGazeCtrl . Looking left and right the neck moves ok, the problem are the eyes that start do crazy movements.

Any idea what could be the problem?
It seems a sign problem somewhere 😅

@pattacini
Copy link
Member Author

@Nicogene give the matrix HN a second look: I might have reported it wrongly.

@Nicogene
Copy link
Member

Ok I will modify and test it

@Nicogene
Copy link
Member

Probably also other part of the library (example) should manage this new head_version

@pattacini pattacini merged commit 98f4b0c into devel Aug 18, 2019
@pattacini pattacini deleted the feat/imu-2.6 branch August 18, 2019 09:14
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.

2 participants