Extend ObjectDetector to read local images through OpenCV

JdeRobot / ObjectDetector

16 stars 14 forks source link

Extend ObjectDetector to read local images through OpenCV #29

Closed jmplaza closed 6 years ago

jmplaza commented 6 years ago

With no dependence on ICE or ROS communication layer

naxvm commented 6 years ago

Sure @jmplaza

The intention is to handle a stream (local video) or a single image/batch of images?

jmplaza commented 6 years ago

stream from local video or local camera

naxvm commented 6 years ago

That can already done streaming from camserver (old cameraserver). Though, that would involve an ICE/ROS (comm) driver. So, I understand the purpose is to read video/camera data directly through OpenCV, right? That would remove the dependency on JdeRobot for this component. Is that correct?

jmplaza commented 6 years ago

Yes, but in a conditional way. The node itself could get images from ROS drivers, from ICE drivers and locally through opencv. Ideally the runtime dependences will depend on the contents of the configuration file (yml), it should only import the corresponding libraries. When using opencv it shouldn't depend on any ROS or ICE library....

naxvm commented 6 years ago

ACK, working on it!

naxvm commented 6 years ago

Finished the integration.

All yours @rkachach :smiley: