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

Change FCL data to Eigen #257

Closed
jcarpent opened this issue Jul 26, 2016 · 4 comments
Closed

Change FCL data to Eigen #257

jcarpent opened this issue Jul 26, 2016 · 4 comments

Comments

@jcarpent
Copy link
Contributor

This issue follows humanoid-path-planner/hpp-pinocchio/issues/10.

It might be useful to pass all the FCL linear algebra type to their eigen counterpart.

@nmansard
Copy link
Contributor

I suggest to keep this discuss for later depending on the choice made by the hpp team.
Any opposition to closing this issue and reopening it in September?

@jcarpent
Copy link
Contributor Author

It is already done in HPP, the change from internal linear algebra to Eigen version. So, from this remark, it is straightforward to move on.

@jmirabel
Copy link
Contributor

jmirabel commented Jul 27, 2016

I agree with Nicolas for delaying the update in Pinocchio to September for two reasons:

  • there are more serious concern before this (doing it or not does not affect us right now),
  • @florent-lamiraux may have a good reason for one choice or another.

@jcarpent
Copy link
Contributor Author

jcarpent commented Aug 6, 2016

Just to mention a few, FCL is moving to Eigen (flexible-collision-library/fcl/pull/150).

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

No branches or pull requests

3 participants