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

add ROS 2 ContentFilteredTopic for Humble Release Note. #2396

Merged
merged 4 commits into from
May 2, 2022

Conversation

fujitatomoya
Copy link
Collaborator

Signed-off-by: Tomoya.Fujita Tomoya.Fujita@sony.com

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Collaborator Author

CC: @wjwwood @ivanpauno @iuhilnehc-ynos

Copy link
Collaborator Author

@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.

since this is release note, i tried to keep it simple.
here is a couple of questions, comments are really welcome.

  • we should add description about currently only available with rclcpp, but rclpy?
  • we should add an explanation about rclcpp::ContentFilterOptions?

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya fujitatomoya added the backport-humble backport at reviewers discretion; from rolling to humble label Apr 27, 2022
@fujitatomoya
Copy link
Collaborator Author

@clalancette @wjwwood could you take a look at this release note? i add the link for content filtering example.

(we need to backport this to humble, labeled so.)

Copy link
Member

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

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

I have a few suggested edits, but lgtm otherwise.

source/Releases/Release-Humble-Hawksbill.rst Outdated Show resolved Hide resolved
Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Collaborator Author

I think it is ready to go! @clalancette @wjwwood requesting final review.

@fujitatomoya fujitatomoya merged commit 29c915b into ros2:rolling May 2, 2022
mergify bot pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)
mergify bot pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)
mergify bot pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)
clalancette pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
clalancette pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
clalancette pushed a commit that referenced this pull request May 2, 2022
* add ROS 2 ContentFilteredTopic for Humble Release Note.

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 29c915b)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble backport at reviewers discretion; from rolling to humble
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants