-
Notifications
You must be signed in to change notification settings - Fork 0
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 sensors to mujoco ergoCub #6
Comments
Trying to add the FT sensors programmatically I found an issue in the converted model: the fixed joints that in the URDF represent the FT location are not present in the MJCF model converted. It seems that the fixed joint gets removed in the processing, the chain does not contain a 'l_shoulder_3 |
Relevant section in mujoco URDF parser: https://github.com/google-deepmind/mujoco/blob/e2c14188adc8f6170aaf03a2f9c3424ab191d629/src/xml/xml_urdf.cc#L436 |
Ah cool, that was exactly the first issue we faced when importing the iCub in Gazebo Classic in 2013 as well, we solved it back in time by adding an option to disable fixed joint lumping gazebosim/sdformat@62e8dd0 and https://osrf-migration.github.io/sdformat-gh-pages/#!/osrf/sdformat/pull-requests/133/page/1 . We could think of ignoring FT sensors for now. |
Ok let's skip FTs for now, looking at mujoco there seems to be no 'fixed' joint too. |
The same problem is present with IMUs, to place them in the URDF we are using fixed joints that vanish in the conversion to mjcf. I think the "correct" way to deal with these should be with |
Nice, also for this we have a Gazebo equivalent: gazebosim/sdformat#262, that then was fixed in gazebosim/sdformat#1148 . |
@vpunithreddy is working on this for ironcub on the https://github.com/ami-iit/mujoco-urdf-loader repo, we can open issues there. |
Thanks @giotherobot I have used iRoncub-Mk3 for reference and have been progressing.
The commits are in the branch |
Sensors
The text was updated successfully, but these errors were encountered: