-
Notifications
You must be signed in to change notification settings - Fork 59
Include openGJK 1.1 #20
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
Conversation
|
@MattiaMontanari, it does not seem to work as well as the original code. Sample tesr: The original code (c3dd37d) converges to f=0: The pull-request code generates "Initial solution: f =nan" and does not converge to 0: |
|
I have pushed a new version as I couldn't find the bug in the previous one... This seems to work now. |
|
It works for the previous case (n=2, id=1) but unfortunately not in general. Here is an example with n=4: Sample tesr: Version 3.5.0 converges to f=0: The pull-request code does not converge to 0: |
update devel
|
Somehow I'm not able to reproduce the results you're showing from this repository (I tried both master v3.5.0 and devel V3.5.1-3). This is what I get from master: If I run openGJK in the new devel branch I get the initial f and final fmin: To get these, I relaxed some of the tollerances (see d605eee). Perhaps I need to configure cmake differently? |
|
Not sure about your not reproducing the 3.5.0 results. |
All done! |
New pull request which addes openGJK 1.1 to Neper devel branch