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

Sync main to 7.0 #87

Closed
wants to merge 8 commits into from
Closed

Sync main to 7.0 #87

wants to merge 8 commits into from

Conversation

MikeWrock
Copy link
Collaborator

@MikeWrock MikeWrock commented Jan 11, 2025

@jeremydbronson I'm hoping to find a better approach to this, but for now we have to make sure main is merged to v7.0 after each release candidate is branched and before testing.

There are a few more updates to main I want to merge in, but before you QA the next rc, lets merge this in

dyackzan and others added 7 commits January 8, 2025 12:54
The MTC plan's computational complexity increases exponentially when
several MoveToPose stages, each with multiple solutions are stacked on
top of each other.

Using PlanCartesianPath with a validation step to ensure no collisions
is a much less computationally complex way to navigate through poses.
…lay-motion

Don't use MTC in Plan and Save Trajectory objective
These new behaviors expose more planning parameters for the Pro RRT
Connect planner.

Replace:
* `SetupMTCMoveToPose` -> `SetupMTCPlanToPose`
* `SetupMTCMoveToJointState` -> `SetupMTCPlanToJointState`

Update planning configuration ports:
* Remove deprecated `planner_interface` port.
* use `keep_orientation` and `keep_orientation_tolerance` ports instead
  of `constraints` port
* Lower `link_padding` where it causes self collisions
Use new Pro RRT SetupMTCPlanTo_ behaviors
@MikeWrock
Copy link
Collaborator Author

This will break functionality, as it depends on https://github.com/PickNikRobotics/moveit_pro/pull/10001

@MikeWrock MikeWrock closed this Jan 13, 2025
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

Successfully merging this pull request may close these issues.

3 participants