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

"k" key doesn't work while teleoperating Spot #9

Open
sskorol opened this issue Dec 10, 2021 · 8 comments
Open

"k" key doesn't work while teleoperating Spot #9

sskorol opened this issue Dec 10, 2021 · 8 comments

Comments

@sskorol
Copy link

sskorol commented Dec 10, 2021

Hi @tarasborov,

I followed the README to launch Gazebo, Spot, inverse, quadruped controller and teleop.
On the teleop tab I have the following instructions:

Screenshot from 2021-12-10 11-03-41

When I tried playing with moving around and holo features, I noticed the Spot doesn't react on "k" / "K" keys. No movements at all. The other keys seem to be working as expected.

BTW, I also noticed the "move forward" -> "I" key is more like a tiptoe. Is there any key to walk forward in a normal way? As backward movement seems more natural.

Thanks

@tarasborov
Copy link
Contributor

K is not involved in a movement control. You can play with speed, body height position, etc.

@sskorol
Copy link
Author

sskorol commented Dec 10, 2021

@tarasborov but it's present in the CLI instruction, isn't it?

Screenshot from 2021-12-10 11-22-45

I also checked https://github.com/SoftServeSAG/teleop_legged_robots/blob/main/src/teleop_legged_robots.py#L56-L73, and there's no "k" key. So probably it'd make sense to remove it from the CLI's help to avoid confusion.

@tarasborov
Copy link
Contributor

You are right. However, on the other hand, we tried to have the same CLI as common teleop has: https://github.com/ros-teleop/teleop_twist_keyboard

@sskorol
Copy link
Author

sskorol commented Dec 10, 2021

But I don't see "k" bindings in this repo as well: https://github.com/ros-teleop/teleop_twist_keyboard/blob/master/teleop_twist_keyboard.py#L40-L59

@tarasborov
Copy link
Contributor

I agree with you. We will take it into account in the next release.

@sskorol
Copy link
Author

sskorol commented Dec 10, 2021

I filed the same issue in their repo as well: ros-teleop/teleop_twist_keyboard#28

Thanks

@sskorol sskorol closed this as completed Dec 10, 2021
@sskorol sskorol reopened this Dec 10, 2021
@sskorol
Copy link
Author

sskorol commented Dec 10, 2021

Feel free to close, when it's released.

@sskorol
Copy link
Author

sskorol commented Dec 13, 2021

Seems like in the original repo they treat "k" as a "stop" action. However, it seems confusing, as there's a note that all the other keys (that are not listed in instructions) behave as a "stop" operation. So for me, it makes no sense to mention it somewhere. I'd rather assign "move back" action to "k", as it would seem more natural if you got used to classic arrow keys.

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

No branches or pull requests

2 participants