Skip to content
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

Problems while loading models in Gazebo 11 #39

Closed
pattacini opened this issue Jul 5, 2020 · 4 comments · Fixed by #40
Closed

Problems while loading models in Gazebo 11 #39

pattacini opened this issue Jul 5, 2020 · 4 comments · Fixed by #40

Comments

@pattacini
Copy link
Member

pattacini commented Jul 5, 2020

Hi,

I've recently started off playing with Gazebo 11 in a Docker Image and I've noticed that there are models provided in this repo that will crumble down to the ground as soon as loaded within the environment.

Here are a couple of examples:

Model working OK
Model NOT working

As @traversaro found out, it might be due to gazebosim/gazebo-classic#2728.

cc @xEnVrE

@diegoferigo
Copy link
Member

I just checked and also my setup is affected by this problem. I confirm that the (no hands) models works fine.

@traversaro
Copy link
Member

The reason why the (no hands) model is working fine is that it is not a URDF file included in an SDF, so it is not affected by gazebosim/gazebo-classic#2728 . On the other hand, the feet_fixed and fixed models are indeed URDF models included in an SDF (see https://github.com/robotology/icub-models/blob/master/fixed_model.sdf.in), so they are indeed affected by gazebosim/gazebo-classic#2728 . As the SDF files are already automatically generated, it should be sufficient to just generate them with version 1.7 if Gazebo 11 is found by CMake.

@pattacini
Copy link
Member Author

pattacini commented Jul 6, 2020

To sum up, we're required to include the patch you recommended here, @traversaro?

@traversaro
Copy link
Member

To sum up, we're required to include the patch you recommended here, @traversaro?

More and less, basically that patch works fine if the sdf files are already in the source repo, while in the case of this repo we need to modify the sdf files that we already automatically generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants