osrf / rmf_demos

Demos to showcase the capabilities of RMF
Apache License 2.0
69 stars 38 forks source link

Change texture.png to more explicative name #142

Closed luca-della-vedova closed 4 years ago

luca-della-vedova commented 4 years ago

Textures with duplicated names cause rendering glitches in ignition, I spotted a texture.png

image

Before the renaming:

image

After:

image

Note that the issue is happening because some of the models in fuel are not updated in the first place and still have their texture named texture.png, specifically from a quick locate texture.png run on my machine:

/home/luca/.gazebo/models/AdjTable/meshes/texture.png
/home/luca/.gazebo/models/MalePatientBed/meshes/texture.png
/home/luca/.gazebo/models/ParkingTrolleyMax/meshes/texture.png
Yadunund commented 4 years ago

Please update changeling before merging. But I guess we can only merge after 30th? @marcoag

marcoag commented 4 years ago

Can merge already I will create a separate branch for the release