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

Dexmg controller #584

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

xieleo5
Copy link
Contributor

@xieleo5 xieleo5 commented Nov 26, 2024

Purpose:
Current IK controller will directly apply the solved joint position to robot joints. For some of the robots, we actually want to use joint action space instead of ee_pos space as action to train BC policies. So we need to add a cache that stores intermediate joint actions.

Changes:

  1. Add the _latest_all_joint_angle_action in WholeBody controller to store the latest joint action input
  2. Add _use_joint_angle_action_input in WholeBody to bypass the IK solving step. We can directly pass joint action in step()
  3. Add _use_joint_angle_action_input in configs.
  4. Load _use_joint_angle_action_input in _validate_composite_controller_specific_config

@kevin-thankyou-lin
Copy link
Contributor

This is a do-not-merge branch right?

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 this pull request may close these issues.

2 participants