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

microRTPS bridge: support Fast-RTPS 1.8.x and above, IDL 4.2 and ROS 2 Dashing #13071

Merged
merged 8 commits into from
Oct 4, 2019

Conversation

TSC21
Copy link
Member

@TSC21 TSC21 commented Oct 2, 2019

Besides a clean up in the templates:

  1. Adds support for Fast-RTPS 1.8.x and the related changes on FastRTPSGen;
  2. Adds an initial support for IDL 4.2, though more work is required in that sense;
  3. Adds support for the ROS2 compatible pubs/subs, which will be used as well in px4_ros_com.

TODO: add deploy of the templates to px4_ros_com.

@TSC21 TSC21 force-pushed the feature/support_RO2_and_IDL4_2 branch from 3ba632b to 025f120 Compare October 2, 2019 10:10
@TSC21
Copy link
Member Author

TSC21 commented Oct 2, 2019

@thomasgubler @jkflying FYI.

@TSC21 TSC21 changed the title microRTPS bridge: support Fast-RTPS 1.8.x and above, IDL 4.2 and ROS 2 Dashing [WIP] microRTPS bridge: support Fast-RTPS 1.8.x and above, IDL 4.2 and ROS 2 Dashing Oct 2, 2019
@TSC21 TSC21 force-pushed the feature/support_RO2_and_IDL4_2 branch from 43a05da to 9c8bb43 Compare October 2, 2019 12:24
@TSC21 TSC21 force-pushed the feature/support_RO2_and_IDL4_2 branch from 4cbafa3 to a7aa89e Compare October 4, 2019 15:46
@TSC21 TSC21 changed the title [WIP] microRTPS bridge: support Fast-RTPS 1.8.x and above, IDL 4.2 and ROS 2 Dashing microRTPS bridge: support Fast-RTPS 1.8.x and above, IDL 4.2 and ROS 2 Dashing Oct 4, 2019
@TSC21 TSC21 merged commit 474c406 into master Oct 4, 2019
@TSC21 TSC21 deleted the feature/support_RO2_and_IDL4_2 branch October 4, 2019 15:56
@hamishwillee
Copy link
Contributor

Cool. Is there a docs change following this?

@TSC21
Copy link
Member Author

TSC21 commented Oct 7, 2019

Cool. Is there a docs change following this?

Yes I am getting those ready.

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.

2 participants