search

si-machines / poli2

High level repository for Poli V2
BSD 2-Clause "Simplified" License
0 stars 7 forks source link

kinova_ros files need to be updated #41

Open asaran opened 6 years ago

asaran commented 6 years ago

When setting up the PoliV2 codebase (Section 2.3, Example 2.5) in the documentation, I ran into errors with catkin_make as below: error: cannot convert ‘moveit::planning_interface::MoveItErrorCode’ to ‘bool’

I could resolve the issue with the link here: https://github.com/ros-planning/moveit/issues/759 I had to replace 'bool' with 'moveit::planning_interface::MoveItErrorCode' in the following 3 files:

  1. ~/catkin_ws/src/kinova_ros/kinova_moveit/kinova_arm_moveit_demo/include/pick_place.h (line 77)
  2. ~/catkin_ws/src/kinova_ros/kinova_moveit/kinova_arm_moveit_demo/include/test_accuracy.h (line 80)
  3. ~/catkin_ws/src/kinova_ros/kinova_moveit/kinova_arm_moveit_demo/src/motion_plan.cpp (line 76)
asaran commented 6 years ago

Modified the kinetic_7dof branch of the fork under si-machines. But do not have access rights for the GT-RAIL fork, from where kinova_ros is pulled automatically when setting up the codebase on Moe. Once the arm_driver fix branch of si-machines/poli2 is merged into master, this should be fixed as the si-machines fork of kinova_ros will be used to setup the codebase on Moe (via the command 'sudo apt-get purge ros-kinetic-dynamixel-workbench-toolbox).