-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add cmake interface #313
Comments
Hi @jbmouret Thanks for sharing the library. It was on my radar a while ago and now I found some applications to explore its use. So far I'm using CMake's By doing the this I was able to link limbo and interpolate using GPs with fixed kernels. However, now I'm struggling to get the NLOpt support to train the hyperparameters, since I get the following warning when trying to build the package that includes limbo: warning: #warning NO NLOpt, and NO Libcmaes: the acquisition function will be optimized by a grid search algorithm (which is usually bad). Please install at least NLOpt or libcmaes to use limbo! Do you have any suggestion on something that I'm probably missing that you internally handle with waf? Thanks! |
Hi First, most of limbo is header-only, so you could simply include the files. Honestly, this is what I would do... I should try to find some time to write some documentation about this. Indeed, you will need to compile limbo with NLOpt and/or CMA-Es for BO. To do so, you need to:
|
Hi, thanks for the reply, I was missing the |
Hi all, I just installed limbo and I would like to use it within my catkin based ros projects. @jbmouret is there any documentation now on this? @mmattamala is there any example publicly available of how you interfaced limbo with your cmake/catkin-based project? Thanks in advance |
Hi @IoannisDadiotis , I have a fork in my account: https://github.com/mmattamala/limbo |
@mmattamala this is very cool. Would you consider making a PR so that the rest of the users can profit from this one? Thanks! |
Yes, for sure. I'll do it now. Thank you! |
Thanks @mmattamala . Do you think just I tried both, but I get the error
My CMAKE_PREFIX_PATH inlcudes the path |
UPDATE: I fixed this by I also had to replace |
Thanks for the follow up @IoannisDadiotis . I was about to ask you if this error was when building limbo itself or when including it in your own package. For the record, just I'll fix the Cmake dependencies I set as required, as @costashatz also pointed in the PR that I open earlier today: #330 |
This is fixed by #330 ! |
We did this in robot_dart and it makes it easier to compile with cmake projects.
The text was updated successfully, but these errors were encountered: