JdeRobot / ObjectDetector

16 stars 14 forks source link

Remove dependencies on ICE (config) when using local camera #31

Closed rkachach closed 6 years ago

rkachach commented 6 years ago

The config module and comm (which I think is part of ICE) are being used/imported when the user is using local camera. As consequence this forces the user to have ICE installed even when using a local video source.

naxvm commented 6 years ago

All fine. Thanks @rkachach!!

naxvm commented 6 years ago

ROS/ICE usage broken from this change (cfg file uncompliant with comm interface).

alexandre2r commented 5 years ago

Hi, I am using the comm interface in my project and it seems broken as @naxvm says. Is it any stable version available? I hardcoded some things in the source code of comm, hoping it will work but I did not test it properly yet... and it is not the best way to use it obviously.