-
Notifications
You must be signed in to change notification settings - Fork 193
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
add torque due to force offset #538
add torque due to force offset #538
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for correcting this!
CI is happy because of style here:
Once that is fixed, we can consider this for inclusion. |
040a014
to
fa63117
Compare
Thanks for pointing that out. I made the change and it looks like CI is happy now. |
fa63117
to
17ee412
Compare
adapted patch by @pac48 . Taken from ros2/geometry2#538 .
I saw this change wasn't yet backported to the humble branch. Is that something that's in the pipeline? |
I can go ahead and do that. |
@Mergifyio backport humble |
✅ Backports have been created
|
* add torque due to force offset (cherry picked from commit 49d4be4)
adapted patch by @pac48 . Taken from ros2/geometry2#538 .
Currently, the
doTransform
function intf2_geometry_msgs.hpp
transforms wrenches by applying pure rotation to the force and torque parts separately. However, this does not correspond to transforming a force vector because the transnational component is ignored. So perhaps the function can be renameddoRotation
or something to that effect. See https://core.ac.uk/download/pdf/154240607.pdf for a reference on transforming force vectors. I made the needed change sodoTransform
now applies a transformation to the given wrench according the method described in the reference I linked.