microsoft / Azure_Kinect_ROS_Driver

A ROS sensor driver for the Azure Kinect Developer Kit.
MIT License
296 stars 222 forks source link

Use ImageTransport::advertiseCamera() #76

Open shuntaraw opened 4 years ago

shuntaraw commented 4 years ago

Is your feature request related to a problem? Please describe. This is a proposal of a minor refactoring to clean up code.

Describe the solution you'd like Replace the pairs of ImageTransport::advertise<Image>() and NodeHandle::advertise<CameraInfo>() in k4a_ros_device.cc with single calls to ImageTransport::advertiseCamera(). If there is a technical reason not to do this, please clarify it in the code comment.

ooeygui commented 2 years ago

This has not been implemented yet.