-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add support for sns dual arm robot #1
Conversation
Add support for sns dual arm robot
Update curobo to v0.7.6
|
}, | ||
} | ||
) | ||
world_cfg = WorldConfig() |
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.
Change to empty WorldConfig, as the world will be get from planning scene.
}, | ||
collision_checker_type=CollisionCheckerType.VOXEL, | ||
collision_checker_type=CollisionCheckerType.MESH, |
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.
Change CollisionCheckerType to mesh
Fix issue no urdf file after build
Copy urdf folder with xrdf folder
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.
I think this file had the same information as sns_description provided, can sns_description used here. Does this file need to be changed when description file changed? If yes, I think it's not easy to maintain at all.
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.
I think this urdf needs to be "pure" urdf. The one with have in sns_description is actually a xacro.
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.
Similar issue, shall we keep acceleration the same as joint_limits.yaml file in sns_moveit_config?
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.
Good point, probably should. But we don't have jerk limit available, and I think the ur's acceleration limits are not really publicly available. The numbers we have are copied from shadow.
Merge with :