-
Notifications
You must be signed in to change notification settings - Fork 110
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
Porting libccd to C++ and using Eigen for math #19
Comments
Hi, Good luck and let me know if you manage to port the whole thing. |
Sorry for the delayed reply. I found some code (e.g., btMprPenetration.h, b3MprPenetration.h) in bullet (probably what you refer to) where the math operation is replaced by bullet's own math code. It seems very similar to what I'd like to do except for I will use Eigen. I created a new repo for this work. For now, it's just a copy of this repo, but will make changes as you suggested so you could see the progress. Thanks! Best, Edit: I'll post here once the porting is in good shape, but you could close this issue in the meantime. |
@jslee02 Did you finish the port? I am curious about the performance improvement. |
libccd is a nice collision detection library for convex shapes. Thanks to @danfis for this great work! I've been using libccd for many years through FCL for the collision detection of our multibody dynamics library DART.
Recently I'm interested in boosting the computational efficiency using SIMD, and would like to apply it to libccd as well. It would be done by replacing the built-in math codes of libccd with Eigen, which is a SIMD-enabled C++ linear algebra library. However, it is unable to switch the built-in math of libccd to Eigen as of now since libccd is written in C. That being said, to use Eigen, libccd needs to be ported to C++, then switch to Eigen. It wouldn't affect on this repository and would be placed in an another repository.
I wonder if @danfis or anyone would be interested in this work, but I'm willing to do it if @danfis don't mind. Please let me know how do you think.
Thanks in advance for any comment.
The text was updated successfully, but these errors were encountered: