search

start-jsk / rtmros_common

OpenRTM - ROS interoperability packages
http://wiki.ros.org/rtmros_common
12 stars 52 forks source link

[hrpsys_ros_bridge/test] remove reverse for samplerobot force-sensors #1108

Open Naoki-Hiraoka opened 3 years ago

Naoki-Hiraoka commented 3 years ago

https://github.com/jsk-ros-pkg/jsk_model_tools/pull/236 のrelease後にmergeしてください.

euscolladaのバグに対応するためのテンポラリの修正としてhttps://github.com/start-jsk/rtmros_common/pull/1080#issuecomment-561094288 https://github.com/start-jsk/rtmros_common/pull/1081 でテストに施された変更を、該当のバグがhttps://github.com/jsk-ros-pkg/jsk_model_tools/pull/236 で修正されるため、戻します。

pazeshun commented 2 years ago

こちらですが、https://github.com/jsk-ros-pkg/jsk_model_tools/pull/236 は0.4.4に含まれ、melodicではrelease済みです。 http://repositories.ros.org/status_page/melodic_default.html?q=euscollada ただ、EOLのkineticに対してはもちろんreleaseされていないので、kinetic以下とmelodic以上でテストコードを分けることになるような気がしています。 http://repositories.ros.org/status_page/kinetic_default.html?q=euscollada もしくは、kinetic以下にeuscollada 0.4.4をソースで入れるかですが、ビルドできるかなどは不明です。 ただし、現在rtmros_commonのmasterでは、melodicにおいてeuslispのテストを行っていないため、この問題は現状回避されています。 個人的には、euslispのテストとしては、現状のkineticまでのテストで当面は大丈夫な気がしていて、melodicのテストを入れる話は、#1107 をのんびり直しながらやっていこうと思っています。