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

Remove "No-IK" simple planner LVS profile #424

Open
marip8 opened this issue Nov 28, 2023 · 1 comment
Open

Remove "No-IK" simple planner LVS profile #424

marip8 opened this issue Nov 28, 2023 · 1 comment

Comments

@marip8
Copy link
Contributor

marip8 commented Nov 28, 2023

As far as I can tell, the SimplePlannerLVSNoIKPlanProfile is now the same as the SimplePlannerLVSPlanProfile version. The "no IK" piece seems to be handled by checking whether the Cartesian waypoint has a seed state associated with it. If it does, then that seed state is used as the joint state with which to interpolate (i.e., "no IK"); if not, IK is performed to get the associated joint state. If this is in fact the case, we should remove the SimplePlannerLVSNoIKPlanProfile class to eliminate confusion

@Levi-Armstrong
Copy link
Contributor

I think it would be find merging the two as log as we add an option to indicate if the seed should be used or run IK.

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

No branches or pull requests

2 participants