-
Notifications
You must be signed in to change notification settings - Fork 55
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
fails to build due to missing H5cpp.h? #41
Comments
Hello @atomoclast , Have you tried installing hdf5 by sudo apt-get install libhdf5-dev I just realized the hdf5 library should be mentioned in CMakeLists.txt as a dependency. It will be included in the next PR. For now, it should work. Please let me know. |
Thank you for the prompt reply! I already have libhdf5-dev installed and it still fails for the same reason... I am trying to build on ros kinetic on 16.04. That may be an issue? |
I did read on this page that you need the H5C++ library... Gonna try and build it from source...because it could be a version issue. What version of hdf5 lib are you using? |
So. I tried to build from source. The most recent release that is. What version are you using? Gonna try and see if I can find the source for H5C++. |
my version is 1.8.11 |
I think openRAVE uses hdf5. In their source, they're importing header files. The issue is that it needs the H5Cpp.h. Which i tried to apt-get with sudo apt-get install libhdf5-cpp-11 Still not working. So it could be a version thing or something I need to build from source. |
Yes. Openrave uses hdf5. But that is hdf5py which can be installed separately. Have you tried to build it from source?? Please try to build from source and let me know. The most probable reason is that the ubuntu and ROS version. We have not imported reuleaux to ros kinetic yet. We are still in indigo. If you can import that you are most welcome. |
So. I just build it with no issue on an indigo/14.04 machine. I'll look into why it doesn't build and fix it, then get back to you! |
@atomoclast, A few months back I managed to get it to build on Kinetic with these changes: It was through trial and error so you may not need all of those changes. |
Looks like it's failing now. It did get farther, but still fails. I'm gonna look into it more. This is more of a side project right now between a few other priorities. Anyways. The output I got is as follows:
|
Do you have a fork of the repo I could take a look at? |
@geoffreychiou, I can make and push one, but all I did was take the current master branch and just update the cmake files and package.xmls with the changes you showed me. My forked changes are here: https://github.com/atomoclast/reuleaux/tree/kinetic-devel |
tried using h5c++ compiler https://github.com/willcbaker/reuleaux/tree/feature/kinetic-devel; aborted compilation after 3 hrs and no errors printed to screen |
@atomoclast, I believe you may have copied the files over directly instead of just pasting over the changes. https://github.com/geoffreychiou/reuleaux/tree/kinetic-build I was able to get your fork to build by adding hdf5_dataset to the target_link_libraries of create_reachability_map and create_capability_map in map_creator/CMakeLists.txt Example: target_link_libraries(create_reachability_map sphere_discretization kinematics -lhdf5 -lhdf5_cpp ${catkin_LIBRARIES})
target_link_libraries(create_capability_map sphere_discretization kinematics -lhdf5 -lhdf5_cpp ${catkin_LIBRARIES}) New: target_link_libraries(create_reachability_map sphere_discretization kinematics -lhdf5 -lhdf5_cpp hdf5_dataset ${catkin_LIBRARIES})
target_link_libraries(create_capability_map sphere_discretization kinematics -lhdf5 -lhdf5_cpp hdf5_dataset ${catkin_LIBRARIES}) I have not tested if it actually runs correctly on kinetic. |
I just tested it. Everything but base_placement plugin builds on my end. The error message i got was:
|
@atomoclast are your sphere_discretization, kinematics and hdf5_Dataset libraries properly built from map_creator package? Have tried creating any maps and visualizing them in rviz? It looks I have to create a kinetic branch to replicate the problems. (It is fine in indigo so far) |
@atomoclast, Have you tried doing a clean build? If you have, push your latest changes to your git and let me see if I can replicate. |
@atomoclast have you been able to solve the problem of running reuleaux on Kinetic yet. Please let me know. I have not been able to find any computer to install kinetic yet. All my work is in indigo and and can't update it now. But if you have not solved it I can borrow a computer and install kinetic there and reproduce your problems. |
Hi guys, sorry for the late reply. Let me look and see, then I'll let you guys know. I have it forked on my page right now. Will update in a few. |
So. I made some changes to the package XML and CMakeLists.txt file, got really close. Seems like there's some issues with HDF5 or something related. I pushed my changes to my fork for you to look at. I can start to delve into some of the issues tomorrow maybe if you don't get a chance to look at it soon. My fork and changes can be found: https://github.com/atomoclast/reuleaux/tree/kinetic-devel
|
@atomoclast modify the link directories in the CMakeLists.txt to
Not the best of ways, but that worked for me. The base_placement plugin compiles as well. However when I add it in rviz panel, it crashes. |
Hello @atomoclast @geoffreychiou @willcbaker @hemangchawla https://github.com/jontromanab/reuleaux/tree/kinetic-devel I am not very familiar with Kinetic. So please test this on your systems, before I ask @akgoins to merge it in the main repository and please let me know about the results. Thank you everybody for your patience. |
Just also ran into this on Kinetic. Will try the fork with the kinetic-devel branch posted by @jontromanab once I find some spare time. Anybody merged those changes with current master? :) |
@skohlbr please let me know if the solution works. |
@jontromanab The kinect-devel branch from your fork works! Thanks! |
Hey @jontromanab! I did follow this, reinstalled hdf5 using this command sudo apt-get install libhdf5-dev and it is the newest version. |
@vajrag Do you have any way to test if the hdf5 library is working separately. Most of the forks from the kinetic branch generally work. Is there any way to see your fork? |
I had the same issue and it turned out I just forgot to change the branch after checkout ;) |
This issue appears again in melodic, any advice? |
As a long time searching on the internet I have just built the package 100% |
huhuhu |
I got this error in melodic. I resolved it by linking both hdf5 and hdf5_cpp. In cmake file i added the following |
This was the error I faced on map_creator |
Hello all,
I'm trying to build these packages, but I'm getting an error on map_creator:
The text was updated successfully, but these errors were encountered: