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

Enable multithreading for OCP + Modify compilation option for the robot #138

Open
3 tasks
jmirabel opened this issue Jan 20, 2025 · 2 comments
Open
3 tasks
Assignees

Comments

@jmirabel
Copy link

Multithreading is implemented using OpenMP. We want to enable OpenMP only inside crocoddyl. This is done using a CMake flag inside crocoddyl.

Tasks to make it work:

  • based on the development Dockerfile, make an experiment Dockerfile that only differs on those points.
    • compiles all code with -03 option and without debug symbols
    • compiles crocoddyl with OpenMP support
  • benchmark the performances of crocoddyl with OpenMP on miyanoura to assess that we are indeed able to have better performances. The test can first be ran in C++. However, it should be tested with Python.
  • Make sure that future experiments uses the experiment image generated using the experiment Dockerfile.
@Kotochleb
Copy link
Contributor

Kotochleb commented Jan 24, 2025

Docker is built now with -O3 and OpenMP enabled in crocoddyl. It is still on RelWithDebInfo instead Release

@jmirabel
Copy link
Author

It is still on RelWithDebInfo instead Release

Why so?

Are you sure your -O3 is not overwritten by RelWithDebInfo, which contains -O2 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants