Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to using new event_handler instead of qos_event. #787

Merged
merged 2 commits into from
Mar 13, 2023

Conversation

clalancette
Copy link
Contributor

@clalancette clalancette commented Dec 20, 2022

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

This is part of ros2/ros2#1361

qos_event was deprecated inside of rclpy in ros2/rclpy#1058 , so use the new API names in this PR.

Part of ros2/ros2#1361

Depends on ros2/rmw#339
Depends on ros2/rcl#1024
Depends on ros2/rclpy#1058

@clalancette clalancette self-assigned this Jan 5, 2023
Copy link
Member

@audrow audrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good with green CI!

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, i will review underlying rclpy and rclcpp as well.

@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 3e9ab14 to 5e0dbf0 Compare January 17, 2023 15:55
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 5e0dbf0 to 775997a Compare January 24, 2023 18:17
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch 2 times, most recently from 505a508 to f0ccc08 Compare February 10, 2023 16:49
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch 2 times, most recently from 4ff95e5 to 3d0c71d Compare March 1, 2023 19:19
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 3d0c71d to b030906 Compare March 7, 2023 18:55
@clalancette clalancette merged commit 57c11d3 into rolling Mar 13, 2023
@delete-merged-branch delete-merged-branch bot deleted the clalancette/inconsistent-topic-event branch March 13, 2023 13:34
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.

4 participants