Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Continuous joints not supported (silently) #57

Closed
henningkayser opened this issue Sep 21, 2023 · 4 comments · Fixed by #59
Closed

Continuous joints not supported (silently) #57

henningkayser opened this issue Sep 21, 2023 · 4 comments · Fixed by #59
Assignees

Comments

@henningkayser
Copy link
Member

When running pick_ik with robot models that have continuous (as unbounded) joints, pick_ik will fail to produce solutions reliably.

Reason: get_random_valid_configuration() will sample states that have inf for joint values, since bounds are set to the numeric limits which the distribution sampler doesn't support

Behavior: FK will silently add nan, inf, or 0 to transforms which in turn fail to compute any useful distance metrics.

We should either check for these cases and warn, or implement a random sampler that supports infinite (or numeric limits) bounds.

@sea-bass
Copy link
Collaborator

sea-bass commented Sep 22, 2023

Oof, nice catch!

What do you think of continuous joints being handled as sampled from 0 to 2*pi radians, and once a solution is achieved snapping it to the closest multiple of 2*pi to its initial seed?

EDIT: The above suggestion assumes revolute joints, which is not necessarily the case.

@MarqRazz
Copy link

FYI the continuous joints on the Kinova are -pi to pi so you might want to decrease the range.

@MarqRazz
Copy link

@sea-bass what version of Studio are you targeting? Could you please update the milestone?

@sea-bass
Copy link
Collaborator

Not necessary to get this in anytime soon since we punted on pick_ik -- can't add a milestone since this PR is not in the Studio repo.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants