-
Notifications
You must be signed in to change notification settings - Fork 4
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
[ROS2] Improve Collision Scene #65
Comments
We should also consider expanding the collision box around the face when not doing the final approach to or retraction from the user's mouth. |
As of ada_ros2#16, the robot rotates with wheelchair rotations, but the collision meshes do not. We need to find a way to handle this. The simplest way would be to just divide the wheelchair and wheelchair collision mesh in half and rotate the upper half with the robot. Although we'd have to see whether MoveIt updates collision objects if the frame they are in updates. |
Re. collision objects rotating with the robot, we should consider attaching some of the collision objects to the robot. This can be done with the ApplyPlanningScene service. That service message's fields |
Right now, our collision scene (#39 ) has several issues:
wheelchair_collision_object
which is meant to expand the wheelchair to account for the user's body/. However, user's have a variety of body types. We have currently implemented awheelchair_collision_object
focused on a large, tall body type. However, if someone who is shorter sits in the wheelchair, their head is in inside thewheelchair_collision_object
which means the robot cannot move to their head. We currently address this by turning off thewheelchair_collision_object
(see [ROS2] Improve Bite Transfer #64 ) but that is dangerous. Instead, we should do one or more of the following:wheelchair_collision_mesh
more granular, so as opposed to having one mesh that expands the wheelchair and adds the user body, we separate this into two meshes. Then, when we move the head mesh in bite transfer, we should also move the body mesh. Then we won't have to turn off the body mesh during transfer.The text was updated successfully, but these errors were encountered: