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

IDL Support (Dashing+ Support) #21

Closed
AndrewJSchoen opened this issue May 12, 2020 · 5 comments
Closed

IDL Support (Dashing+ Support) #21

AndrewJSchoen opened this issue May 12, 2020 · 5 comments

Comments

@AndrewJSchoen
Copy link

I wanted to separate out this specific issue to prevent overloading the topic in the main Dashing support thread. Specifically, this is a discussion for the planned feature of IDL support, required by ROS2 Dashing and later versions.

Given the discussions there, there seem to be a couple of issues that need resolving, one major one being the choice of either using static compilation or introspection. I would be interested in any opinions that people have about that, as I look myself into how best to go about this.

@lelongg
Copy link
Contributor

lelongg commented May 20, 2020

For reference here is the link to the start of this discussion: #19

Looking at the state of RTPS and IDL support on rust, here is a pair of interesting developments: rtps-gen and rtps-rs.

It doesn't look ready but using these could prevent having to copy messages from Rust to C before serialization (and vice versa for deserialization).

@nnarain
Copy link
Contributor

nnarain commented Jun 12, 2020

Hi. Just wanted to say I'm also interested in this and giving it a shot. Current changes on the dashing-plus branch in my fork.

Ported the IDL generator and made some fixes for the rcl API. Publish and subscribe examples are working. Need to clean it up though.

Also, I'm specifically building against Dashing. Figured I'd do Eloquent and Foxy too (Not sure to what extent things need to change there).

@nnarain
Copy link
Contributor

nnarain commented Jul 7, 2021

Wanted to note that Galactic has introduced some new tools for rosidl. Might be easier now to integrate with cargo.

@ghost
Copy link

ghost commented Oct 24, 2021

@nnarain is your branch able to publish and subscribe geometry_msgs like twist and pose? Possibly pointcloud msgs as well? Huge help if can as reported here: #37

And have you upgraded to Galactic or are you still on Dashing?

@nnmm
Copy link
Contributor

nnmm commented Mar 18, 2022

I think this can be closed now – after #80, IDL support should be practically complete, at least in Foxy+.

@nnmm nnmm closed this as completed Mar 18, 2022
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

No branches or pull requests

4 participants