gazebosim / gz-sim

Open source robotics simulator. The latest version of Gazebo.
https://gazebosim.org
Apache License 2.0
621 stars 251 forks source link

[Err] [SDFFeatures.cc:843] The geometry element of collision [collision] couldn't be created #2325

Open PerFrivik opened 4 months ago

PerFrivik commented 4 months ago

Desired behavior

[Err] [SDFFeatures.cc:843] The geometry element of collision [collision] couldn't be created

I don't think that this error is very constructive or helpful as I am trying to add a plugin from asv_wave_sim into PX4 and don't know what the issue is. My boat just fell through the waves and I was wondering if it had to do with this Error, but again I don't know what is causing this error.

I put this under feature request, as I don't think that it is a bug, but rather that the pipeline probably needs some improving, I could be wrong!

If anyone knows more, I would be super grateful for any information/help, thanks!

iche033 commented 4 months ago

That error msg is misleading and may not the actual cause for the boat falling through waves.

The msg was changed to a debug msg (from error) in harmonic (https://github.com/gazebosim/gz-physics/pull/531) and backported to garden (https://github.com/gazebosim/gz-physics/pull/581).

srmainwaring commented 4 months ago

That error msg is misleading and may not the actual cause for the boat falling through waves.

@iche033 it's definitely not the reason for the hydrodynamics plugin not working. There'll be something not quite configured right in the setup of the PX4 example. Will continue to track in: https://github.com/srmainwaring/asv_wave_sim/issues/157.

The message doesn't appear to impact creating collision meshes, or having them collide, or be used for other purposes. I'm not exactly sure what is not working correctly from a practical sense (maybe writing out to SDF as part of a snapshopt of the world). In any case when the models and plugins are set up correctly the simulation does not appear to be impacted, so it's mainly a noisy warning that may be nice to be able to optionally suppress.