Skip to content

Add qos parameter for wait_for_message function (backport #2903)#2907

Merged
fujitatomoya merged 1 commit intohumblefrom
mergify/bp/humble/pr-2903
Jul 17, 2025
Merged

Add qos parameter for wait_for_message function (backport #2903)#2907
fujitatomoya merged 1 commit intohumblefrom
mergify/bp/humble/pr-2903

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Jul 16, 2025

Description

A small change to include the QoS settings as one of the parameters for rclcpp::wait_for_message. This is useful in cases where we want just to get the last published message on the topic, rather than always waiting for a new message.

Fixes # (issue)

Is this user-facing behavior change?

Did you use Generative AI?

No

Additional Information


This is an automatic backport of pull request #2903 done by [Mergify](https://mergify.com).

Signed-off-by: Sriharsha Ghanta <ghanta1996@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
(cherry picked from commit 2fcef70)
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented Jul 16, 2025

Pulls: #2907
Gist: https://gist.githubusercontent.com/fujitatomoya/0b7ca6fbf91b626916e2433f9c5d6e23/raw/c4814743034b82c44bbb24c2a92cad0893365633/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: humble
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16503

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fujitatomoya
Copy link
Copy Markdown
Collaborator

https://ci.ros2.org/job/ci_linux-rhel/3781/ are unrelated.

@fujitatomoya fujitatomoya merged commit 8e19cba into humble Jul 17, 2025
3 checks passed
@ahcorde ahcorde deleted the mergify/bp/humble/pr-2903 branch July 17, 2025 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants