Topic/small improvments

rohanpsingh / mc_mujoco

MuJoCo interface for mc-rtc

https://arxiv.org/abs/2209.00274

BSD 2-Clause "Simplified" License

81 stars 18 forks source link

Closed gergondet closed 3 years ago

gergondet commented 3 years ago

Hi @rohanpsingh

This PR fixes the following issues:

add missing return statement (fix wrong codegen with some compilers)
introduce MUJOCO_ROOT_DIR in CMake script to avoid relying on the hard-coded position
set a reasonable default for MUJOCO_KEY_PATH (the location advised by Mujoco)

rohanpsingh commented 3 years ago

Thanks a lot for the improvement!