search

MRPT / mvsim

Vehicle and mobile robotics simulator. C++ & Python API. Use it as a standalone application or via ROS 1 or ROS 2
https://mvsimulator.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
251 stars 40 forks source link

Implement first subscribe, then advertise #21

Closed jlblancoc closed 1 year ago

jlblancoc commented 1 year ago

Fixing this to-do:

https://github.com/MRPT/mvsim/blob/3fc84437e7727da2aeaac63f36eeb1941513427c/modules/comms/src/Comms/Server.cpp#L223-L227

will avoid spurious test errors:

1: [18:44:55.1650|DEBUG|mvsim::Server] Subscription request for topic /r1/laser1_scan'
1: [18:44:55.1654|DEBUG|mvsim::Client] Advertising topic `/r1/laser1_scan` [mvsim_msgs.ObservationLidar2D] on endpoint `tcp://0.0.0.0:43105`
jlblancoc commented 1 year ago

Fixed by 02b74e260d7a2522e20a9f532967f1383cd2cfb5