Closed hsd-dev closed 6 years ago
+1 this works for me, travis looks good and code change looks good. It turns out i still have merge privileges, should i go ahead and merge?
Speaking of deps, godel still depends on @Jmeyer1292's descartes "godel_refactor_omp" branch. There is a newer branch at https://github.com/ros-industrial-consortium/descartes but that fails in the build with the following error:
godel/godel_process_planning/src/generate_motion_plan.cpp:79:47: error: no matching function for call to ‘descartes_planner::DAGSearch::run(std::vector<double>&)’
double cost = search.run(process_start_costs);
If there is interest in using the newer version of descartes with godel, let me know and i will update. Thanks!
I am going to merge this unless there are any objections...
Fix for
Cannot locate rosdep definition for [godel_path_planning]
and forcannot convert ‘moveit::planning_interface::MoveItErrorCode’ to ‘bool’