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

Joystick Controller Communication port #83

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jdcaporale
Copy link
Contributor

Work done by @chandravaran in a different repo with very minor edits (comments and name changes).

Adding joystick controls Python scripts.
Added features:

  • Configurable controllers, JoystickAbstract
  • Logitech and Xbox controller example classes
  • Example script for sending LCM kinematic twist command [x_dot, omega]

Issues:

  • Some of the maps seem backward or redundant in how they are defined and used. We may want to have human-readable names that get mapped to the hex addresses instead of the reverse. Or at least maybe some master gamepad input list e.g. left-stick, south-button (instead of "A" or "X"), assuming some minimal button set, and if it is missing a mapping function that replaces it. Overgeneralizing may be a problem here so mostly we would just want XBox, Logitech, Playstation, Generic or something.

Todo:

  • Add the controller_command.lcm struct
  • Fix the mappings to be more useful to a general audience
  • Perhaps draw up an example where more buttons are used e.g. expanding the twist command to be able to jump with "A" button
  • Make an example script that runs with a BehaviorIOin a simple robot.
  • YAML configuration of the controllers

@jdcaporale jdcaporale added enhancement New feature or request priority-med Medium priority issues labels Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-med Medium priority issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant