search

ZebraDevs / fetch_grasp_suggestion

GNU Lesser General Public License v3.0
7 stars 6 forks source link

Moveit gives error during Catkin build #5

Open virraaj opened 5 years ago

virraaj commented 5 years ago

Hi, Thanks for this package. I am trying to run it in Ubuntu 18.04 with ros melodic. I get the following error during catkin build

src/fetch_grasp_suggestion/include/fetch_grasp_suggestion/executor.h:18:10: fatal error: moveit/move_group_interface/move_group.h: No such file or directory
 #include <moveit/move_group_interface/move_group.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I did install the moveit using 

sudo -E apt-get install ros-melodic-moveit

Can you let me know where I am going wrong.

moriarty commented 5 years ago

Sorry- this was an intern project, and i haven’t found a new intern to update the project to 18.04 yet.

moriarty commented 5 years ago

Pull requests welcome if you can fix the error, but I’ll be out of the office for the next week.

virraaj commented 5 years ago

Hi, thanks for the reply. I could not yet fix the error. It is most probably because of move_group.h file which is not available in the directory of move_group_interface. So, I think it could be some upgrade issue from moveit . I will try to get some way around if possible or close the issue in the next week.

moriarty commented 5 years ago

https://answers.ros.org/question/306640/include-move_group_interface/

dekent commented 5 years ago

Hi, sorry for the delayed reply. It looks like you're using the master branch, which was developed for ROS indigo. There's a melodic devel branch that should fix the problems you're seeing. Let me know if you're still seeing issues building on 18.04 using the melodic-devel branch.

Ideally the master branch should be removed, now that the fetch platform is supported on both indigo and melodic. @moriarty is it cool to delete the master branch and set a new default? I'm not sure what your best practices for organizing branches are.

moriarty commented 5 years ago

Yep, lets delete master and have:

indigo-devel & melodic-devel

Melodic will be the default, Indigo is EOL and won’t be getting any new features or support, but we’ll leave the branch around.