Skip to content
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

kinova_ros files need to be updated #41

Open
asaran opened this issue Apr 24, 2018 · 1 comment
Open

kinova_ros files need to be updated #41

asaran opened this issue Apr 24, 2018 · 1 comment

Comments

@asaran
Copy link

asaran commented Apr 24, 2018

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: moveit/moveit#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
Copy link
Author

asaran commented Apr 27, 2018

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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant