ros2 / rclpy

rclpy (ROS Client Library for Python)
Apache License 2.0
310 stars 227 forks source link

No way to set rosout QoS for a Node #1368

Open rg-gravis opened 1 month ago

rg-gravis commented 1 month ago

Feature request

Feature description

The rclcpp NodeOptions class has a rosout_qos field that allows you to set the QoS policy for when the node publishes to /rosout as part of ROS logging. The rclpy Node constructor has no such option, so python users are stuck with the default rosout_qos, which is unsuitable for some applications (we're experiencing dropped log messages that can be resolved for cpp nodes by adjusting the rosout_qos).

Implementation considerations

An additional parameter in the Node constructor would work perfectly well, and would just need to get passed to rcl here

cottsay commented 1 month ago

Hi there, this seems like a reasonable feature to add. Please consider opening a pull request to implement it!

fujitatomoya commented 4 weeks ago

@rg-gravis you can try https://github.com/ros2/rclpy/pull/1376.