vikashplus / robohive

A unified framework for robot learning
https://sites.google.com/view/robohive
Apache License 2.0
489 stars 82 forks source link

Support tele-op for Franka Kitchen environments ("vel" mode) #142

Open sriramsk1999 opened 5 months ago

sriramsk1999 commented 5 months ago

Changes Summary:

Motivation:

Currently, the teleop code only supports "pos" mode environments, where the expected input action is the qpos coordinates. Other environments, like the Franka Kitchen envs, are "vel" mode - where input action is the velocity of each joint.

NOTE: Normalizing the qvel does not work, the values are too small and the end effector never reaches the teleop target. Is there some issue with how the normalization is called?

vikashplus commented 4 months ago

I haven't tested in cases you are using it now. This might be a good chance for me to test those. I'm traveling this week. I'll be able to take a look next week.

sriramsk1999 commented 4 months ago

Bump on this PR. Even though the teleop works, it would be good to know if the implementation is correct, especially since it doesn't work when I normalize the velocities.

vikashplus commented 4 months ago

Let me know when its ready for a merge and I'll do a thorough review.

sriramsk1999 commented 4 months ago

Ready for review!