-
Notifications
You must be signed in to change notification settings - Fork 23
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
URDF models have non-consistent joint limits #48
Comments
Please note that the limits in the robots-configuration files are sometimes even inconsistent among different robots, even if the different robots are the same version of the iCub (i.e. we should check the limits before copying them). Having said that, I think we should have actual hardware limits in the URDF. In the |
Once we have the "actual" limits for the iCub 2.5, we can update the file https://github.com/robotology-playground/icub-model-generator/blob/master/simmechanics/data/icub2_5/ICUB_2-5_BB_joint_parameters.csv to propagate this changes to all generated models. |
what about finalizing this table? can we reorganize the wiki for the different iCub models? |
👍 for finalizing that table in the current location, differentiating it for iCub version if necessary. |
Recently the iCub Manual has been updated with a page about the joint limits. We could use the |
I would go for CAD RoM, while the software limits should go in the Gazebo configuration files. However it is commont practice to use the URDF limits for planning movements, so we should make sure no one is actually doing that in the lab, and instead is reading the joint limits from the robot at runtime. |
This is somehow related to #5 .
The limits displayed in this file are different from those defined in https://github.com/robotology/robots-configuration/tree/devel .
For example the
l_hip_roll
has different limits from those defined in this file.We should even discuss whether to use the "soft" or the "hardware" limits. (https://github.com/robotology/robots-configuration/blob/devel/iCubGenova04/hardware/motorControl/left_leg-eb6-j0_3-mc.xml vs https://github.com/robotology/robots-configuration/blob/devel/iCubGenova04/hardware/mechanicals/left_leg-eb6-j0_3-mec.xml)
The text was updated successfully, but these errors were encountered: