search

ros-naoqi / pepper_meshes_installer

Other
0 stars 1 forks source link

Normals of LShoulderRoll #1

Closed asherikov closed 4 years ago

asherikov commented 7 years ago

Normals of 'LShoulderRoll' link seem to be messed up. I had to fix them with blender to display link properly.

mikaelarguedas commented 7 years ago

thanks for reporting it: duplicate of https://github.com/ros-naoqi/pepper_meshes/issues/1 @suryaambrose @nlyubova can any of you recompute the normals in blender and create a new installer with the updated meshes?

nlyubova commented 7 years ago

@asherikov thank you for your comment! Could you do a Pull Request?

mikaelarguedas commented 7 years ago

Unfortunately no, it should be fixed in the original blender file of the robot that is proprietary so that it is fixed for any future mesh export. So only maintainers within Softbank can do it and generate a new installer with the updated meshes.

asherikov commented 7 years ago

Yes, the licence prohibits derivative works. Moreover, it would be better if a more experienced person performed the conversion: My version of blender generates slightly different collada files -- is this an issue? Is it necessary to update *.stl file?

mikaelarguedas commented 7 years ago

I don't think it the change in collada will be an issue as long as it's supported by all versions of Rviz and Gazebo (so in the end OGRE collada importer) where this package is released. Yes I think the stl should be modified as well. But the script handling that is importing the collada meshes, decimating them and them exporting them as stl so it should be straight forward.