-
Notifications
You must be signed in to change notification settings - Fork 2
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
Moveit humble #88
base: main
Are you sure you want to change the base?
Moveit humble #88
Conversation
Test Results236 tests 222 ✅ 33m 25s ⏱️ Results for commit f5f8193. ♻️ This comment has been updated with latest results. |
Scenario Results8 tests 8 ✅ 14m 10s ⏱️ Results for commit 5440da5. ♻️ This comment has been updated with latest results. |
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.
Successfully tested on my machine. First draft looks good, some cleanup is needed
scenario_execution_ros/scenario_execution_ros/actions/move_to_joint_pose.py
Outdated
Show resolved
Hide resolved
simulation/gazebo/arm_sim_scenario/launch/sim_moveit_scenario_launch.py
Outdated
Show resolved
Hide resolved
simulation/gazebo/arm_sim_scenario/launch/sim_moveit_scenario_launch.py
Outdated
Show resolved
Hide resolved
self.moveit2.planner_id = "RRTConnectkConfigDefault" | ||
|
||
# Scale down velocity and acceleration of joints (percentage of maximum) | ||
self.moveit2.max_velocity = 0.5 |
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.
for now ok, but this should become a parameter of the action
callback_group=ReentrantCallbackGroup() | ||
) | ||
|
||
self.moveit2.planner_id = "RRTConnectkConfigDefault" |
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.
for now ok, but this should become a parameter of the action
scenario_execution_moveit/scenario_execution_moveit/actions/move_to_joint_pose.py
Outdated
Show resolved
Hide resolved
scenario_execution_moveit/scenario_execution_moveit/actions/move_to_pose.py
Outdated
Show resolved
Hide resolved
scenario_execution_moveit/scenario_execution_moveit/actions/move_gripper.py
Outdated
Show resolved
Hide resolved
scenario_execution_moveit/scenario_execution_moveit/actions/move_to_joint_pose.py
Outdated
Show resolved
Hide resolved
scenario_execution_moveit/scenario_execution_moveit/actions/move_to_pose.py
Outdated
Show resolved
Hide resolved
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.
what's the status here?
|
||
scenario example_moveit: | ||
wx200: robotic_arm with: | ||
keep(it.namespace == 'wx200') |
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.
could you please update this to the "modern" syntax, i.e., remove keep
Description
moveit2 lib for scenario execution to control robotics manipulator
Pull request checklist
make check
) pass locallyPull request type
Current behavior
No moveit2 support to control manipulators
New behavior
moveit2 support
How to test
First Install dependencies.
second:
third run: