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

Support Connext key fields #119

Open
superware opened this issue May 7, 2023 · 2 comments
Open

Support Connext key fields #119

superware opened this issue May 7, 2023 · 2 comments
Labels

Comments

@superware
Copy link

Hello,

I have a working ROS2 foxy ecosystem running Connext and came to the point where I need Connext key fields support. I'm looking for the simplest way to modify ROS2 or rmw_connextdds and allow the underlaying Connext RMW to have some of the fields defined as keys.

Most of the related rmw implementation seem to be in rmw_connext_dynamic_cpp/src/utility_templates.hpp which in conjunction with DDS_StructMember.is_key seems technically possible but I don't have the necessary architectural knowledge to workaround.

Any help will be greatly appreciated.

@clalancette
Copy link
Contributor

There is no current plan to work on this. This is actually a pretty complicated feature request, so if you were interested in pursuing it please start by opening a design document REP on https://github.com/ros-infrastructure/rep .

For now I'll put this on the backlog.

@superware
Copy link
Author

I'm not referring to ROS support for key fields, but rather to somehow add the is_key bit in Connext RMW typesupport. Which issues would rise in ROS if the RMW will internally handle keyed topics?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants