search

leggedrobotics / raisimLib

RAISIM, A PHYSICS ENGINE FOR ROBOTICS AND AI RESEARCH
http://www.raisim.com
327 stars 50 forks source link

Bug fix in getDenseJacobian and getDenseRotationalJacobian methods of ArticulatedSystem. #44

Closed nash169 closed 4 years ago

nash169 commented 4 years ago

Small fix in the methods to get the dense Jacobian. I guess that the second for loop should iterate till the dimension of the sparse Jacobian and not the dense one.

jhwangbo commented 4 years ago

Thank you Nash! There are some methods that I don't use and I didn't put this in my unit tests.

I can't merge a pull request because of a copyright issue but I'll recreate it on my code. I appreciate your contribution!