-
Notifications
You must be signed in to change notification settings - Fork 126
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
Review primitive types #190
Comments
For these changes the existing design docs need to be considered too. There are specific mappings for DDS as well as Python which don't have an equivalent if these would be removed. Therefore I think this needs further consideration and shouldn't be done for the beta 1. |
I'm not advocating to get this in the beta. It would be great but maybe a bit premature. |
I never really replied to #190 (comment):
Note: the fact that ROS1 consider |
@mikaelarguedas update design accordingly |
@mikaelarguedas need to update design to keep these types and offer flexibility for users to request a specific structure to receive them |
It would be nice to have a type (perhaps It would also almost entirely clear up ros2/ros2#509 |
Relevant discussion at https://discourse.ros.org/t/discussion-about-how-to-add-a-new-primitive-types-to-rosidl-msg/4607/3 as well. |
With the migration to use |
This is a meta-ticket for type deprecationg and primitive types supported is ROS 2 IDL.
The text was updated successfully, but these errors were encountered: