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
Adding EarlyStopped flag in GJK #371
Conversation
Here the distance has a meaning as a lower distance bound. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comment: the idea of using NaN values is for propagation in the computational graph. For the end-user, no contact should be added in case a separating hyperplane is found.
Importantly, this PR solves a bug occurring in Pinocchio in debug mode. |
for more information, see https://pre-commit.ci
When GJK early stops, the closest points
p1
andp2
make no sense, so we can simply set them tonan
.Q: What should we do about
distance
? For now it is set to the distance between the origin and the separating hyperplane, but we could set it tonan
like the closest points.