RethinkRobotics-opensource / sns_ik

Saturation in the Null Space (SNS) Inverse Kinematic Library
82 stars 41 forks source link

Corrected SawyerModel defined for unit tests #100

Closed robodreamer closed 6 years ago

robodreamer commented 6 years ago

This commit updates sawyer model defined for tests. I found out that the previous model definition was incorrect. Because of that, LIN_SOLVE_RESIDUAL_TOL had to be set very high (1e1). Now, with the correct model definition, the LIN_SOLVE_RESIDUAL_TOL is set to a reasonable number (1e-8) which makes better sense.

Note that with this change, the position solver test results do not look as good as before but it might be because of how the tests were formulated. We will revisit the tests later.