robomaster-oss / rmoss_core

RoboMaster OSS中的基础项目,为RoboMaster提供通用基础功能模块包,如相机模块,弹道运动模块等。
Apache License 2.0
105 stars 17 forks source link

Add camera_info topic to `rmoss_cam` #36

Closed Ericsii closed 1 year ago

Ericsii commented 1 year ago

Feature request

Feature description

可以考虑在rmoss_cam中加入sensor_msgs::cameraInfo消息的topic

Implementation considerations

gezp commented 1 year ago

想问一下,这么做的应用场景有哪些?camera_info有必要和image同频率发布吗?

Ericsii commented 1 year ago

已改用分离的publisher来发布camera_info信息。加入这个publisher是为了兼容其他大部分的ros开源项目。

gezp commented 1 year ago

因为rmoss_cam中的CamServer是设计和CamClient搭配使用的,就像CameraPublisherCameraSubscriber搭配使用一样。当然为了提高兼容性确实是一个考虑的点,我觉得可以加一个bool参数use_image_transport_camera_publisher来确定是使用原始的模式,还是使用CameraPublisher模式来兼容其他模块。

Ericsii commented 1 year ago

好的,这种形式应该可以。我改一下这个PR

Ericsii commented 1 year ago

@gezp 已经修改PR

gezp commented 1 year ago

done