ros2 / rmw_fastrtps

Implementation of the ROS Middleware (rmw) Interface using eProsima's Fast RTPS.
Apache License 2.0
155 stars 117 forks source link

ASYNCHRONOUS is not reported by Fast DDS Monitor #758

Closed Krzo99 closed 4 months ago

Krzo99 commented 4 months ago

Bug report

Steps to reproduce issue

Following this example, using the exactly same code and provided .xml, and env:

FASTRTPS_DEFAULT_PROFILES_FILE=/my_path/SyncAsync.xml
RMW_FASTRTPS_USE_QOS_FROM_XML=1
RMW_IMPLEMENTATION=rmw_fastrtps_cpp

Create a publisher and subscriber. Code compiles, and works. However, if I check in FastDDS Monitor, the publishing mode is still set to "SYNCHRONOUS"

image

Expected behavior

The publishing mode would be correctly set to ASYNCHRONOUS, as provided in .xml config file.

Actual behavior

The publishing mode is always set to SYNCHRONOUS, regardless of .xml config

Additional information

The SyncAsync.xml file is found correctly, as changing any tags gives errors when running the node.

Mario-DL commented 4 months ago

Hi @Krzo99 Thanks for the report. The issue has to do with the fact that Fast DDS Monitor is not reporting ASYNCHRONOUS, but it does not mean that the publish_mode is not correctly set with the configuration you provide. We will come back when we have a fix. In the meantime I would suggest changing the title to ASYNCHRONOUS is not reported by Fast DDS Monitor

clalancette commented 4 months ago

Given that this looks like a bug in Fast-DDS Monitor, and not in rmw_fastrtps, I'm going to go ahead and close this. Thanks for the report.