jsk-ros-pkg / jsk_model_tools

JSK model utilities
https://github.com/jsk-ros-pkg/jsk_model_tools
BSD 3-Clause "New" or "Revised" License
5 stars 27 forks source link

[eusurdf] Use collada_urdf instead of collada_urdf_jsk_patch #184

Closed wkentaro closed 7 years ago

k-okada commented 7 years ago

I think we need collada_urdf_jsk_patch for some reason @YoheiKakiuchi

YoheiKakiuchi commented 7 years ago

https://github.com/jsk-ros-pkg/jsk_common/pull/557 @mmurooka Do we still need collada_urdf_jsk_patch ?

mmurooka commented 7 years ago

Last check was Hydro @ May, 2014. cf. https://github.com/start-jsk/rtmros_gazebo/issues/48#issuecomment-41999914 I'll check again with new environment, but if supporting hydro, I think that keeping to use collada_urdf_jsk_patch is safe.

wkentaro commented 7 years ago

I confirmed using collada_to_urdf in collada_urdf package does not work to spawn models on Gazebo correctly, and collada_urdf_to_jsk_patch is still required. I tested with room73b2 models.

wkentaro commented 7 years ago

So closing at this time.

k-okada commented 7 years ago

so we need to wait for assimp 3.2 ? @yoheikakiuchi https://launchpad.net/ubuntu/+source/assimp

◉ Kei Okada

On Thu, Sep 1, 2016 at 1:01 PM, Kentaro Wada notifications@github.com wrote:

So closing at this time.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/jsk-ros-pkg/jsk_model_tools/pull/184#issuecomment-243970177, or mute the thread https://github.com/notifications/unsubscribe-auth/AAeG3BUivXvid0HRTfZB7iOayJZ55uzKks5qlk41gaJpZM4JwWb8 .