Unverified Commit 08c5060b authored by Octi Zhang's avatar Octi Zhang Committed by GitHub

Adapts A and D button bindings in the keyboard device (#910)

# Description

While developing some teleoperation methods, I discovered that the
keyboard y-axis was moving in a non-intuitive direction. It turned out
that the +ve and -ve y-axis are bound to keys (D, A) instead of (A, D).
This MR fixes the issue in both keyboard device documentation
instructions to follow the right-hand rule.

## Type of change

- Bug fix (non-breaking change which fixes an issue)

## Screenshots

![Screenshot from 2024-08-30
14-57-26](https://github.com/user-attachments/assets/5f29054e-f0e8-4b8e-bb90-49f3403a1e08)


## Checklist

- [X] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [X] I have made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [X] I have updated the changelog and the corresponding version in the
extension's `config/extension.toml` file
- [X] I have added my name to the `CONTRIBUTORS.md` or my name already
exists there
parent b01447d6
[package]
# Note: Semantic Versioning is used: https://semver.org/
version = "0.22.6"
version = "0.22.7"
# Description
title = "Isaac Lab framework for Robot Learning"
......
Changelog
---------
0.22.7 (2024-09-05)
~~~~~~~~~~~~~~~~~~~
Fixed
^^^^^
* Adapted the ``A`` and ``D`` button bindings inside :meth:`omni.isaac.lab.device.Se3Keyboard` to make them now
more-intuitive to control the y-axis motion based on the right-hand rule.
0.22.6 (2024-08-29)
~~~~~~~~~~~~~~~~~~~
......
......@@ -170,9 +170,9 @@ class Se3Keyboard(DeviceBase):
# x-axis (forward)
"W": np.asarray([1.0, 0.0, 0.0]) * self.pos_sensitivity,
"S": np.asarray([-1.0, 0.0, 0.0]) * self.pos_sensitivity,
# y-axis (right-left)
"D": np.asarray([0.0, 1.0, 0.0]) * self.pos_sensitivity,
"A": np.asarray([0.0, -1.0, 0.0]) * self.pos_sensitivity,
# y-axis (left-right)
"A": np.asarray([0.0, 1.0, 0.0]) * self.pos_sensitivity,
"D": np.asarray([0.0, -1.0, 0.0]) * self.pos_sensitivity,
# z-axis (up-down)
"Q": np.asarray([0.0, 0.0, 1.0]) * self.pos_sensitivity,
"E": np.asarray([0.0, 0.0, -1.0]) * self.pos_sensitivity,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment