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

Bring in libcmaes as a dependency #16433

Closed
hongkai-dai opened this issue Jan 25, 2022 · 5 comments · Fixed by #16441
Closed

Bring in libcmaes as a dependency #16433

hongkai-dai opened this issue Jan 25, 2022 · 5 comments · Fixed by #16441
Assignees
Labels
component: build system Bazel, CMake, dependencies, memory checkers, linters priority: low

Comments

@hongkai-dai
Copy link
Contributor

hongkai-dai commented Jan 25, 2022

Now that libcmaes has the Apache 2.0 license, I think it is ready to bring it as a Drake dependency

@hongkai-dai hongkai-dai added the component: build system Bazel, CMake, dependencies, memory checkers, linters label Jan 25, 2022
@jwnimmer-tri jwnimmer-tri changed the title Bring in libcmaes as a dependency. Bring in libcmaes as a dependency Jan 26, 2022
@jwnimmer-tri
Copy link
Collaborator

jwnimmer-tri commented Jan 26, 2022

@hongkai-dai will provide a small sample function (or program, or test) that calls cmaes, and then @jwnimmer-tri will open a PR with a sufficient BUILD file (+ releated) to get that working in Drake.

@jwnimmer-tri
Copy link
Collaborator

A good sample program to use would be https://github.com/CMA-ES/libcmaes/blob/master/examples/sample-code.cc.

@jwnimmer-tri
Copy link
Collaborator

@hongkai-dai for tracking purposes, could you document in this issue which Drake feature this dependency will be in service of? Ideally, that new feature would have its own overall Drake tracking issue that you could just link to here.

@jwnimmer-tri
Copy link
Collaborator

@hongkai-dai Do we have any timeline for using this CMA-ES library in Drake? We added it 18 months ago as a speculative dependency ("dev" code) and haven't touched it since then (other than babysitting it to keep it up-to-date).

Now that #17231 is nearing completion, I either need to fix the CMA-ES to be hidden-safe or else remove it from Drake. (We could bring it back later again if we want to, reverting the removal.)

@hongkai-dai
Copy link
Contributor Author

I ended up not using CMA-ES. I think we can close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: build system Bazel, CMake, dependencies, memory checkers, linters priority: low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants