HorizonRobotics / SocialRobot

Apache License 2.0
71 stars 20 forks source link

Replace irobot create with kuka youbot #84

Closed Jialn closed 5 years ago

Jialn commented 5 years ago

Changes:

Motivation:

youbot: image

youbot doing goal task trained by SAC: youbot

Jialn commented 5 years ago

If create is replaced, please make sure to update all the places using or mentioning irobot create throughout the whole project. For example:

https://github.com/HorizonRobotics/SocialRobot/blob/1bfac7bab9779e3126034c8e63142845a3daa534/python/social_bot/envs/grocery_ground.py#L522-L523

in groucery_ground.py need to be updated.

Comments and Read-me and tests are also updated.

Jialn commented 5 years ago

Nice addition Jiangtao. I didn't know that irobot doesn't have touch sensor. May worth leaving it in if we have plans to add touch sensor in the future.

irobot does not have a touch sensor, but have a gazebo ray sensor (infrared range sensor) which serves similar function. We can also add that easily to pioneer.

Actually model of irobot create is inside gazebo's model database and we did not keep a copy in our repo. So you can see I barely remove anything. It's very easy to add it back by just adding a simple config in agent_cfg.jason if needed in the future.