andyzeng / visual-pushing-grasping

Train robotic agents to learn to plan pushing and grasping actions for manipulation with deep reinforcement learning.
http://vpg.cs.princeton.edu/
BSD 2-Clause "Simplified" License
932 stars 313 forks source link

UR 3.X support #11

Open robotlinker opened 5 years ago

robotlinker commented 5 years ago

Hi I am using a UR5 robot version 3.5.1 to test the package and get errors, looks like it is only tested with UR Software version 1.8? Any suppor for Version 3.X using ROS ur_modern_driver? Thanks!

WwYyFan commented 5 years ago

@robotlinker Hi,I also encountered this problem. Can you solve it now?

nro-bot commented 5 years ago

...What errors are you encountering?

I was able to get it working with Version 3.x, there was a little finickiness with the parse_tcp_state (assert message type 16), which I hacked around (by duplicating any tcp_socket.recv lines). This is probably doing something like flushing out data until it's indexed into the right place, but I'm moving on for now. A more robust TCP URx library can be found at https://github.com/SintefManufacturing/python-urx ; I haven't really dug into the differences.

kwang4372 commented 5 years ago

Hi, I have a problem with 'Robot' class init. i modify a line ('assert(robot message type == 16) to 20. ( i have ur3 3.5) but, i encounter another problem. At return of parse_tcp_state_data, I have a error "struct.error : unpack requires a string argument of length 8" Please, give me a help 스크린샷, 2019-07-31 18-12-30

스크린샷, 2019-07-31 18-13-18

liuxiangdi commented 5 years ago

Hi, I have a problem with 'Robot' class init. i modify a line ('assert(robot message type == 16) to 20. ( i have ur3 3.5) but, i encounter another problem. At return of parse_tcp_state_data, I have a error "struct.error : unpack requires a string argument of length 8" Please, give me a help 스크린샷, 2019-07-31 18-12-30

스크린샷, 2019-07-31 18-13-18

Hi, Have you solve this ploblem?, I am using UR5 version3.5.4, i meet the same problem, I tried use UR script a API and it works, But I still can not use this code

Abdul-UOiT commented 5 years ago

@liuxiangdi how are you trying to connect with the real ur5 do you use ROS ur_modern_driver?

Abdul-UOiT commented 5 years ago

can anyone here explain how to connect with real robot? should it be connected through ros vrep bridge

liuxiangdi commented 5 years ago

@liuxiangdi how are you trying to connect with the real ur5 do you use ROS ur_modern_driver?

no, finally I use urx-python package for UR control (the parser code in this repo don't suit newest UR version) [(https://github.com/SintefManufacturing/python-urx)] for realsense, you can use code in this repo(just change an API in realsense reading code)

Lightwingy commented 5 years ago

@liuxiangdi , can you run the code in the real robot? If you can, how do you use urx-python package for UR control?

liuxiangdi commented 5 years ago

@liuxiangdi , can you run the code in the real robot? If you can, how do you use urx-python package for UR control?

you can use code in this repo(need rewrite the UR parser part, in my robot version 3.5, this code parser is old version) or you can use URX-python, and just delete all control code in this repo then change to urx-python. It is not very difficult because this paper just have 4 direction basic move, you can use urx-python for control test, it is very easy because it provide high level API.e. I run a similary task in UR5, i use code in this repo first, it can run but i did not test the pushing and grasping task because i don't have robotQ for grasping.

Lightwingy commented 5 years ago

@liuxiangdi Thank you!