-
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
Use the generated model in Gazebo #52
Comments
As discussed f2f with @DanielePucci , @francesco-romano and @fiorisi , for unblocking this issue the plan is to automatically change the inertia of the small links using the |
I remember @nunoguedelha some while ago performed some test with implicit damping without much luck |
Yes, but I think that the enabling the implicit spring damping should decrease the amount to which the inertia should be increased. |
Regarding the point "Handling the small inertias." in the checklist, we changed the inertia of some of the smallest robot links. This is to be ported into the URDF generation scripts. The impacted links are listed below, with the respective inertia delta. All the changed link inertia have been set to
|
@diegoferigo @traversaro Edited the checklist in the initial comment of this issue. I think we can close this issue. |
Apparently my tests weren't that good. Changing the PIDs does the job! Setting 0 for the derivative and the integral part seems to be enough. |
iCub models meetingThis afternoon @nunoguedelha, @diegoferigo, @traversaro and I had a meeting about how to use iCub models. These are the outcomes of our meeting: Definition of the new workflow for models
TODO list:
|
Edited meeting minutes #52 (comment): converted "TODO list" to a check list. |
@nunoguedelha can you please update the issue? |
The only issue still open related to this issue is #56 . However, that issue does not prevent to use the models in Gazebo, and so I think we can close this issue. |
In this issue I want to track and recap the efforts done (mainly by @nunoguedelha) and what's missing in the generation chain to let these model work in Gazebo.
icub-models
repository (Updated models after icub-model-generator PR 49 icub-models#2)model.config
for all the robots (Install all the generated models icub-models#3)<sensor>
tag). Right now gazebo crashes without giving any error (Gazebo crashes without throwing any error if we add the accelerometers to the URDF #54)icub-gazebo-wholebody
repo (initial PR Allow using the configurations shipped in this project specifying the robot model name robotology-legacy/icub-gazebo-wholebody#12 and fix Fix/icub model config and fixation to world robotology-legacy/icub-gazebo-wholebody#15)icub-models
(such models have a default pose and joint configuration and appear hovering 1mm above the ground) => Raw iCub model from icub-models repo cannot be used directly #79Other pending issues imported from robotology/icub-models#7:
cc @nunoguedelha @traversaro @DanielePucci
The text was updated successfully, but these errors were encountered: