-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add functions to convert shape_msgs/SolidPrimitive <-> euslisp object #640
Conversation
Thank you for your review. I named these functions following existing functions: I pushed an additional commit to reflect your advice. |
please check travis errors
|
Thank you for your advice. Cause of errorI have checked the travis error carefully, I found following:
What I should doFor kinetic and melodic tests, I think we should wait for jsk_pcl_ros to be released on http://packages.ros.org/ros-testing/ubuntu |
this issue is related to jsk-ros-pkg/jsk_visualization#765 . |
Please add depends to |
please refer jsk-ros-pkg/jsk_visualization#699 (comment) jsk-ros-pkg/jsk_visualization#644 (comment) for jsk_visualizaiotn case. this is more complex than jsk_roseus, but it seems we do not want jsk_visualization depends on jsk_recognition. |
Thank you for your teaching.
I have misunderstood the package system. Also, from these comments (#641 (comment), jsk-ros-pkg/jsk_visualization#699 (comment) and jsk-ros-pkg/jsk_visualization#644 (comment)), I understand that we should not add large packages dependencies. I'll be careful from now on. |
what i waant to tell you is that jsk_pcl_ros is not released in ros-testing is because of jsk_visualization. |
@knorth55 Now, I am trying to understand the system of ROS Buildfarm. In my understanding, the current situation is as follows:
|
|
@pazeshun |
I added functions to convert
shape_msgs/SolidPrimitive
<->euslisp object
shape_msgs/SolidPrimitive
is used at collision-object-publisher.l, for example.https://github.com/jsk-ros-pkg/jsk_pr2eus/blob/master/pr2eus_moveit/euslisp/collision-object-publisher.l#L80-L85
If this pull request is merged, I will replace the above code with cube->shape-msg() functions.