Skip to content

Add Python implementation for a Chain filter#213

Merged
ahcorde merged 2 commits intoros2:rollingfrom
EsipovPA:add_chain_filter_python
Oct 13, 2025
Merged

Add Python implementation for a Chain filter#213
ahcorde merged 2 commits intoros2:rollingfrom
EsipovPA:add_chain_filter_python

Conversation

@EsipovPA
Copy link
Copy Markdown
Contributor

@EsipovPA EsipovPA commented Oct 9, 2025

Description

Add Chain filter implementation for Python

Fixes #204

Is this user-facing behavior change?

Yes.
This change provides the functionality of a Chain message filter for Python users.

Did you use Generative AI?

No, I did not.

Additional Information

@EsipovPA EsipovPA force-pushed the add_chain_filter_python branch from 03729f7 to 4cc9092 Compare October 10, 2025 09:05
Comment on lines +1 to +2
# Copyright 2015, Open Source Robotics Foundation, Inc. All rights reserved.
# Copyright 2015, Martin Llofriu. All rights reserved.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Copyright 2015, Open Source Robotics Foundation, Inc. All rights reserved.
# Copyright 2015, Martin Llofriu. All rights reserved.
# Copyright 2025, Open Source Robotics Foundation, Inc. All rights reserved.

from rclpy.time import Time
from std_msgs.msg import String

PKG = 'message_filters'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove

@classmethod
def setUpClass(cls):
rclpy.init()
cls.node = rclpy.create_node('my_node', namespace='/my_ns')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
cls.node = rclpy.create_node('my_node', namespace='/my_ns')
cls.node = rclpy.create_node('TestChainNode', namespace='/my_ns')

@EsipovPA EsipovPA force-pushed the add_chain_filter_python branch from 4cc9092 to edf6a99 Compare October 13, 2025 09:56
Signed-off-by: EsipovPA <esipov.p@mail.ru>
…ilters_chain.py

Signed-off-by: EsipovPA <esipov.p@mail.ru>
@EsipovPA EsipovPA force-pushed the add_chain_filter_python branch from edf6a99 to 612983f Compare October 13, 2025 09:57
@EsipovPA EsipovPA requested a review from ahcorde October 13, 2025 11:43
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Oct 13, 2025

Pulls: #213
Gist: https://gist.githubusercontent.com/ahcorde/3f404e05170369a2addd247d34fd32fd/raw/32d79833fddbaf186b251942cf392c0d22df71b1/ros2.repos
BUILD args: --packages-above-and-dependencies message_filters
TEST args: --packages-above message_filters
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17283

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

@ahcorde ahcorde merged commit 0617849 into ros2:rolling Oct 13, 2025
2 checks passed
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Oct 13, 2025

https://github.com/Mergifyio backport kilted jazzy humble

@mergify
Copy link
Copy Markdown

mergify bot commented Oct 13, 2025

backport kilted jazzy humble

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Oct 13, 2025
Signed-off-by: EsipovPA <esipov.p@mail.ru>
(cherry picked from commit 0617849)
mergify bot pushed a commit that referenced this pull request Oct 13, 2025
Signed-off-by: EsipovPA <esipov.p@mail.ru>
(cherry picked from commit 0617849)

# Conflicts:
#	CMakeLists.txt
#	src/message_filters/__init__.py
mergify bot pushed a commit that referenced this pull request Oct 13, 2025
Signed-off-by: EsipovPA <esipov.p@mail.ru>
(cherry picked from commit 0617849)

# Conflicts:
#	CMakeLists.txt
#	src/message_filters/__init__.py
ahcorde pushed a commit that referenced this pull request Oct 14, 2025
(cherry picked from commit 0617849)

Signed-off-by: EsipovPA <esipov.p@mail.ru>
Co-authored-by: Pavel Esipov <38457822+EsipovPA@users.noreply.github.com>
ahcorde added a commit that referenced this pull request Oct 15, 2025
* Add Python implementation for a Chain filter (#213)

Signed-off-by: EsipovPA <esipov.p@mail.ru>
(cherry picked from commit 0617849)


Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Co-authored-by: Pavel Esipov <38457822+EsipovPA@users.noreply.github.com>
Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
ahcorde added a commit that referenced this pull request Oct 15, 2025
* Add Python implementation for a Chain filter (#213)

Signed-off-by: EsipovPA <esipov.p@mail.ru>
(cherry picked from commit 0617849)
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Co-authored-by: Pavel Esipov <38457822+EsipovPA@users.noreply.github.com>
Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
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.

Add Python implementation for the Chain filter

2 participants