Skip to content

[2605] WheelAnimTool: remove Eigen dependency + fix rotation - #184

Merged
jhanca-robotecai merged 4 commits into
o3de-2605from
jh/wheel_anim_eigen
Jun 30, 2026
Merged

jhanca-robotecai merged 4 commits into
o3de-2605from
jh/wheel_anim_eigen

Conversation

@jhanca-robotecai

@jhanca-robotecai jhanca-robotecai commented Jun 29, 2026 •

Copy link
Copy Markdown
Collaborator

What does this PR do?

  • Fix duplicate wheel position loop in InitJacobian: the position-gathering loop ran twice, causing meanWheelPosition to accumulate 2x the correct sum and shifting all relative wheel positions
  • Auto-compensate for mirrored wheel meshes: added per-wheel m_wheelAxisSign computed at activation by comparing each wheel entity's spin axis direction against the robot body's spin axis
  • Remove Eigen dependency, replace with AZ::Math:
    • Eigen::MatrixXd -> AZ::MatrixMxN
    • Eigen::VectorXd -> AZ::VectorN
    • Eigen::Vector3d / Eigen::Vector2d -> AZ::Vector3 / AZ::Vector2
    • Matrix-vector multiply -> AZ::VectorMatrixMultiply

Removing Eigen was done to solve RobotecAI/agentic-mobile-manipulator#31

The version of the Gem was updated to 2.0.0, as 1.x.y serie is used for o3de-2505 branch.


How was this PR tested?

Tested using https://github.com/RobotecAI/agentic-mobile-manipulator/ demo project
The Robot needed updates of the controller:
Screenshot from 2026-06-29 12-59-02


Screenshots / Logs / Supporting Evidence (if applicable)

wheel_anim.mp4

Committer Checklist

Please confirm the following before marking this PR as ready for review:

  • Code changes are well-documented
  • Tests have been added or updated
  • Code owners file and CI config were updated if new Gem was added
  • The code is formatted according to the project's style guide
  • The version number has been updated according to the contributing guidelines

Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>
Signed-off-by: Jan Hanca <jan.hanca@robotec.ai>

@knicked knicked left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I applied the suggested updates in RobotecAI/agentic-mobile-manipulator#32 and tested the changes, LGTM

@jhanca-robotecai
jhanca-robotecai merged commit b814f8d into o3de-2605 Jun 30, 2026
1 check passed
@jhanca-robotecai
jhanca-robotecai deleted the jh/wheel_anim_eigen branch June 30, 2026 07:08
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.

3 participants