-
Notifications
You must be signed in to change notification settings - Fork 7
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
dbase_grasp_planner compilation error #2
Comments
I'm not very familiar with dbase_grasp_planner, but will have a closer look soon and see if I can help. One thing would be good to sort out first: You may not need to use my fork if you are not using grasp_planning_graspit or any of the other packages in graspit-pkgs. Is this the case? My understanding is as well that the migration for this repository here to catkin/cmake is ongoing work/TODO at the moment, is that right, @jvarley?. It still has to be made compatible with the new cmake setup of graspit, which doesn't support catkin (yet?). The original graspit source was meant to stay catkin-free I think. One way to use the new graspit version from here would be to include the original graspit source with externalproject_add. Or, include it as regular cmake (not catkin) dependency. In the latter case, the cmake build in graspit doesn't come with a graspitConfig.cmake yet, but I think that this is going to be merged as PR soon there as well, or what's you opinion, @jvarley? |
Oh yeah I just had another look, with the version in this repo you are actually using an old verison of graspit, pulling it from svn (unless the one on svn has been updated, I haven't inspected the source on the svn now). In any case, if you have the new graspit version (from my fork) in your catkin ws as well, that's bound to have plenty of compatibility issues. Maybe this issue can help to push forward with merging these packages with the new graspit version... :) |
Hi Jenny! Thanks for taking a look at this.
I'd like to use the urdf2* packages. Are those dependent on the rest of the metapackage, including your fork of graspit? Could I remove the rest of the metapackage and just keep them?
I removed graspit_simulator's copy of graspit and replaced it with your fork. |
Hi Jack, urdf2* packages don't depend on grasp_planning_graspit or graspit packages. Anyway in this case, at this point it makes no real difference whether you use original graspit or my fork. I see what you did is replace the You could try to remove all other packages in this repo except dbase_grasp_planner and write a new CMakeLists.txt for it, for which you'd also have to look at dbase_grasp_planning.pro. Would be a bit of fiddling though. |
Jenny, Thanks for clarifying those dependencies. I spent enough time fruitlessly fiddling before posting this issue, that I'm reluctant to start anew :).
I wanted to automate the generation and storage of grasps for a set of new objects. dbase_grasp_planner interfaces with Willow Garage's household_objects_db, a Postgres database of objects and grasps for a PR2 gripper. It seemed like my best option when I found it, but I now think that an hdf5 file would be better suited to my application. I'm actually in contact with @jvarley regarding adding relevant functionality to graspit. So, for now, that's the option I'll be pursuing, and this issue is no longer of immediate importance to me. :) |
Hi all,
I'm trying to compile dbase_grasp_planner from source, but am running into the following errors:
I've modified the source code so that it connects to my local household_objects_db. I've also made the following modifications for compatibility with current tools and libraries:
Two other things to note are that I'm using Jennifer Buehler's fork of GraspIt, and that I've also ported graspit_dbase_tasks to catkin (and changed its database connection parameters; this package compiles without issue).
All that being said, I'm stumped by the error I'm getting here. GraspableBody is defined in graspit's body.h; the getDBModel() function is dependent on a compiler flag, but this flag is set to true in the CMakeLists.txt for this version of GraspIt (see line 305).
Any help would be much appreciated!
Thanks,
Jack
CMakeLists.txt
The text was updated successfully, but these errors were encountered: