search

plusone-robotics / moveit_simple

A wrapper around MoveIt that enables more traditional industrial robot programming.
Apache License 2.0
15 stars 12 forks source link

Added prettyprint header to robot to fix build error #65

Closed geoffreychiou closed 6 years ago

geoffreychiou commented 6 years ago

CI was failing with the latest ROS release. Build failed locally so I re-ran an old pipeline.

I guess they removed the << overload for vectors?

Here is error, it should be fixed now.

Errors     << moveit_simple:make /root/catkin_ws/logs/moveit_simple/build.make.000.log
In file included from /opt/ros/kinetic/include/ros/ros.h:40:0,
                 from /root/catkin_ws/src/moveit_simple/moveit_simple/src/robot.cpp:25:
/root/catkin_ws/src/moveit_simple/moveit_simple/src/robot.cpp: In constructor ‘moveit_simple::Robot::Robot(const ros::NodeHandle&, const string&, const string&)’:
/root/catkin_ws/src/moveit_simple/moveit_simple/src/robot.cpp:54:23: error: no match for ‘operator<<’ (operand types are ‘std::basic_ostream<char>’ and ‘const std::vector<std::__cxx11::basic_string<char> >’)
     << ", and tool: " << robot_model_ptr_->getLinkModelNames());

@joshuaplusone please review.

geoffreychiou commented 6 years ago

@jonlwowski012

geoffreychiou commented 6 years ago

@shaun-edwards please add @jonlwowski012 to the group.

jonlwowski012 commented 6 years ago

:+1:

130s commented 6 years ago

I guess they removed the << overload for vectors?

@geoffreychiou which module you think changed?

geoffreychiou commented 6 years ago

@geoffreychiou which module you think changed?

@130s, it looks like they took it out of rosparam_handler in the latest release. We just got lucky they had the overload in there. Prettyprint was supposed to take care of that functionality.