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

Use pro rrt mtc behaviors #9

Closed

Conversation

dyackzan
Copy link

@dyackzan dyackzan commented Jan 3, 2025

Use new SetupMTCMoveTo_ProRRT behaviors

Replace:
* `SetupMTCMoveToPose` -> `SetupMTCMoveToPoseProRRT`
* `SetupMTCMoveToJointState` -> `SetupMTCMoveToJointStateProRRT`

The new behaviors expose more planning parameters for the Pro RRT
Connect planner.

Will convert from draft once https://github.com/PickNikRobotics/moveit_studio/pull/10001, which adds the new behaviors in core goes in

git-subtree-dir: src/fanuc_sim
git-subtree-split: 0b1e054ccb6a4fa20b2658ad318ebc1acfd64bb6
git-subtree-dir: src/lab_sim
git-subtree-split: 2ee0461a27dc2165441ce9362a81db44303177d4
git-subtree-dir: src/moveit_pro_kinova_configs
git-subtree-split: 178a37c6d6bfd703ea6018266aa79872802f31b9
…367b7

git-subtree-dir: src/moveit_pro_mobile_manipulation
git-subtree-split: e5367b79e08113218f72804beda69ebe94a2b175
git-subtree-dir: src/moveit_pro_ur_configs
git-subtree-split: fe3c9a7f6f0a27cf881a34aa551ed6b2b876c8af
… commit 3db6acb

git-subtree-dir: src/moveit_pro_mobile_manipulation/ridgeback
git-subtree-split: 3db6acb098765ff28952642fbab6dc3261efe973
git-subtree-dir: src/ros2_robotiq_gripper
git-subtree-split: 77e60927cd5cdcc7c75b8837ac3104b826a4bb99
…om commit 77e6092

git-subtree-dir: src/external_dependencies/ros2_robotiq_gripper
git-subtree-split: 77e60927cd5cdcc7c75b8837ac3104b826a4bb99
…db6acb

git-subtree-dir: src/external_dependencies/ridgeback
git-subtree-split: 3db6acb098765ff28952642fbab6dc3261efe973
git-subtree-dir: src/external_dependencies/serial
git-subtree-split: d8d160678aa0b31cdf467c052b954fa287cc6cdf
…om 77e6092..2ff8545

2ff8545 update urdf mesh filenames to use package path (#50) (#68)
93ecde7 Replace fake_sensor_commands with 'mock_sensor_commands (#62)
REVERT: 77e6092 update urdf mesh filenames to use package path (#50)
REVERT: 27ecc36 Replace fake_sensor_commands with 'mock_sensor_commands (#61)
REVERT: 5f0c39a Remove mimic tags and rename `sim_ignition` to `sim_gazebo` (#54)

git-subtree-dir: src/external_dependencies/ros2_robotiq_gripper
git-subtree-split: 2ff85455d4b9f973c4b0bab1ce95fb09367f0d26
git-subtree-dir: src/picknik_registration
git-subtree-split: 220e5c96aac04bf4879b4106f16238fee2a46825
git-subtree-dir: src/example_behaviors
git-subtree-split: d1451e331ba27aee0af9bbaafaefc8bf37a48aca
@dyackzan dyackzan requested review from marioprats and sjahr January 3, 2025 02:32
@dyackzan dyackzan force-pushed the use-pro-rrt-mtc-behaviors branch 2 times, most recently from 3d27c49 to 23e52b1 Compare January 8, 2025 18:37
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
@dyackzan dyackzan force-pushed the use-pro-rrt-mtc-behaviors branch from db2bcdf to 91268bc Compare January 9, 2025 13:57
@dyackzan dyackzan closed this Jan 9, 2025
@dyackzan dyackzan deleted the use-pro-rrt-mtc-behaviors branch January 9, 2025 23:22
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.

9 participants