search

ros2 / rclpy

rclpy (ROS Client Library for Python)
Apache License 2.0
289 stars 224 forks source link

Allow specifying qos (backport #1225) #1274

Open mergify[bot] opened 5 months ago

mergify[bot] commented 5 months ago

This function didn't work for transient local publishers.

This is a breaking change because the argument order is changed. However with the future in mind, this order makes most sense to me as all first arguments are passed directly to the subscription.

If you like the original order I can change this.

This is an automatic backport of pull request #1225 done by Mergify.

fujitatomoya commented 5 months ago

@Timple just FYI, this is the backport of https://github.com/ros2/rclpy/pull/1225 for iron. i will cherry-pick this to https://github.com/ros2/rclpy/pull/1272.

fujitatomoya commented 5 months ago

CI: