search

dilipkumar25 / see3cam

ROS driver for the e-consystems See3CAM_Stereo (Tara) camera
https://www.e-consystems.com/3D-USB-stereo-camera.asp
21 stars 20 forks source link

Tara_launch: Tried to advertise a service that is already advertised in this node [/stereo/set_camera_info] #13

Closed Changliu52 closed 6 years ago

Changliu52 commented 7 years ago

It seems the tara node is trying to advertise two [/stereo/set_camera_info] rosservice, rather than [stereo/left/set_camera_info] and [stereo/right/set_camera_info]

Please update this, because this is very important for running the stereoCalibration in ros.

Kind Regards, Chang

jveitchmichaelis commented 6 years ago

Hi @Changliu52 - please check out our fork here; https://github.com/i3drobotics/deimos-ros

It doesn't look like PRs are being accepted here, hence the shameless plug :)

I've made a number of modifications to the package (including fixing this issue). It's designed for our Deimos stereo system, but should be compatible with your camera.

void-robotics commented 6 years ago

Is this package still being supported? I'm also seeing the same double advertisement message.

The deimos one sort of worked for me; some rendering issues in ROS kinetic

jveitchmichaelis commented 6 years ago

@nathangeorge1 What rendering issues are you seeing? I can take a look if you like. Otherwise you could probably take inspiration from the Deimos package and use it to modify this one if it seems to work better, though we haven't altered the bulk of the code, just fixed some of the topics.

@hellozhouuu care to comment on the thumbs down? If you're having issues I'd be happy to help.

void-robotics commented 6 years ago

@jveitchmichaelis sure; let me open an issue on your GitHub repo

Changliu52 commented 6 years ago

Great! Thanks