-
Notifications
You must be signed in to change notification settings - Fork 0
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
HPP version for Kinodynamic planner #1
Comments
Hi, sorry I took time to answer. Sanghyun's version is supposed to be similar to the one in the docker. @pFernbach can you confirm ? |
Sanghyun uses newer libraries for instance he uses hpp-pinocchio, which correspond to the new version of HPP. |
I am pretty sure it is not the case, since rbprm only works since yesterday with pinocchio |
Sanghyun is supposed to work on the same branch as you (dynamicPlanner) of rbprm which do not use hpp-pinocchio. There may be few commits of difference between you depending of when you pulled the branch but I do not know why he use hpp-pinocchio. |
There are the dependencies used in Sanghyun's package hpp-timeoptimization (from the CMakeLists): But for the code I installed in the docker, i.e. Pierre code, "git describe" gives: Maybe he is using different versions for generating contact plans and for its program...? |
The Makefile that I made should install the following versions You can check the number of the last commit. What is sure is that this version of hpp-rbprm do not require hpp-pinocchio neither hpp version >= 4 . Anyway, we are working on porting hpp-rbprm with the latest release of hpp and we will make a release of hpp-rbprm as soon as it's done (2 weeks max). I will update the docker once this is done. |
Hi,
This week, I quickly tried to install Sanghyun's software to replace Justin's one in the docker. However, the current version of the kinodynamic planner is based on an old version of HPP so I got conflicts. I know that @stonneau is updating hpp-rbprm but is it the static version or also the kinodynamic one?
Best,
Mathieu.
The text was updated successfully, but these errors were encountered: