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

is_move_playing returns False while movement is not finished #383

Closed
FabienDanieau opened this issue Sep 26, 2024 · 0 comments · Fixed by #418
Closed

is_move_playing returns False while movement is not finished #383

FabienDanieau opened this issue Sep 26, 2024 · 0 comments · Fixed by #418
Assignees
Labels
bug Something isn't working

Comments

@FabienDanieau
Copy link
Contributor

The bug can be reproduced with this code https://github.com/pollen-robotics/demo_month/blob/main/task1-sdk/box-moving/box_moving.py and the gazebo simulator. It needs to be run 5 to 10 times. Then the issue stays. The issue being that the gripper is closed to soon here (before the movement from one "box" to another).

is_move_playing returns False because the received status is UNKNOWN (

state = self._get_move_state(id)
)

This status is received from the SDK server here : https://github.com/pollen-robotics/reachy2_sdk_server/blob/a610c73c2d20ac9a9eadfc28fc4ec545cb602af1/reachy_sdk_server/reachy_sdk_server/grpc_server/goto.py#L344

which call the pollen_goto ROS node. @RemiFabre any idea?

@FabienDanieau FabienDanieau added the bug Something isn't working label Sep 26, 2024
@FabienDanieau FabienDanieau self-assigned this Sep 26, 2024
@RemiFabre RemiFabre self-assigned this Sep 26, 2024
@FabienDanieau FabienDanieau linked a pull request Oct 8, 2024 that will close this issue
FabienDanieau added a commit that referenced this issue Oct 9, 2024
…rns-false-while-movement-is-not-finished

bug #383: remove is_goto_plyaing function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants