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

[ROS2] MoveAwayFromMouth Actions #53

Closed
amalnanavati opened this issue Aug 29, 2023 · 0 comments · Fixed by #72
Closed

[ROS2] MoveAwayFromMouth Actions #53

amalnanavati opened this issue Aug 29, 2023 · 0 comments · Fixed by #72
Assignees

Comments

@amalnanavati
Copy link
Contributor

Currently, when moving away from the mouth, the web app calls the vanilla MoveAbovePlate and MoveToRestingPosition. However, the trouble with this is:

  1. When near the user's mouth, the robot is in collision with the expanded wheelchair_collision object. Therefore, no vanilla action will work. We need an action that will allow collisions with the wheelchair_collision object, initially (e.g., as the robot moves back to the staging location) and then disallows collisions as it either moves above the plate or moves to the resting position.
  2. Using vanilla actions prevents us from doing any other customizations of robot motion when moving away from the mouth (e.g., lowering the speed).

Note that currently MoveToMouth #42 permanently allows collisions with the wheelchair_collision object to account for this, which is dangerous. Therefore, addressing this is a top priority.

@amalnanavati amalnanavati self-assigned this Aug 29, 2023
@amalnanavati amalnanavati changed the title MoveAwayFromMouth Actions [ROS2] MoveAwayFromMouth Actions Aug 30, 2023
@amalnanavati amalnanavati linked a pull request Aug 31, 2023 that will close this issue
8 tasks
@amalnanavati amalnanavati removed a link to a pull request Sep 2, 2023
8 tasks
@amalnanavati amalnanavati linked a pull request Sep 2, 2023 that will close this issue
8 tasks
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 a pull request may close this issue.

1 participant