Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

SetAxis implementation incorrect for DART #2505

Closed
osrf-migration opened this issue Aug 7, 2018 · 2 comments
Closed

SetAxis implementation incorrect for DART #2505

osrf-migration opened this issue Aug 7, 2018 · 2 comments
Labels
bug Something isn't working major

Comments

@osrf-migration
Copy link

Original report (archived issue) by Michael Grey (Bitbucket: mxgrey, GitHub: mxgrey).


While trying to get a better understanding of the reference frame for joint axes in SDF, I noticed that the implementation of SetAxis() for DARTHingeJoint has a bug. Specifically, this line shouldn't be in there. That transforms the joint axis into the parent link frame, but we just want it to be expressed in the frame that it's already in.

It seems this has gone unnoticed because that transform from the parent link frame is set to an identity matrix at the point in time that this function is called by the SDF loading routine, so it has no effect. However, if a user calls this function later, after the whole SDF has been loaded, then the axis will be set incorrectly if there's any relative rotation between the parent link and joint poses.

I already have a fix and regression test written for this. I'm opening this issue so that I can tag the regression test with an issue number.

@osrf-migration
Copy link
Author

Original comment by Michael Grey (Bitbucket: mxgrey, GitHub: mxgrey).


See pull request #3005 for a fix.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


  • changed state from "new" to "resolved"

pull request #3005

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working major
Projects
None yet
Development

No branches or pull requests

1 participant