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

Migrate sensor_msgs utilities to ROS 2 #3

Closed
wjwwood opened this issue Jun 17, 2015 · 6 comments
Closed

Migrate sensor_msgs utilities to ROS 2 #3

wjwwood opened this issue Jun 17, 2015 · 6 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@wjwwood
Copy link
Member

wjwwood commented Jun 17, 2015

ROS 1's sensor_msgs contains C++ headers and Python modules that provide some utilities for the generated message code data types, e.g. filling an image message or iterating over a point cloud.

I avoided bring that with the current migration effort. We should decide if we want to host those utilities elsewhere of if we want them in the sensor_interfaces package, similar to how it is laid out in ROS 1. It seems like it would be best if those things were in a sensor_interfaces_utilities package instead, but maybe there is a good reason for the current colocation in ROS 1.

TODO: update these entries in the migrated messages once this is done:

Also look into avoiding things like valid encoding values being captured in C++ headers and instead capture them in constants in the messages. Again, there might be a good reason for this, but it smells to me.

@wjwwood wjwwood added the enhancement New feature or request label Jun 17, 2015
@clalancette
Copy link
Contributor

Just FYI, I have done parts of this as needed for the cartographer integration effort. I think we ended up deciding to just keep these utilities as part of the same sensor_msgs package for ease of use/porting from ROS1. That being said, I haven't done all of the utilities, just the ones I've needed (point cloud iterators and distortion models), so we should leave this open until we decide what to do about the rest.

@mikaelarguedas
Copy link
Member

related PRs: pointcloud2 iterators: #34 and distortion models #31.

Also if we decide to lint these migrated files we should also make sure they comply with our developer guide (doesn't seem to be the case AFAICT)

@tfoote tfoote added the help wanted Extra attention is needed label Feb 22, 2018
@sagniknitr
Copy link

Any updates on regarding migrating these utilities ? We need fill_image.h for our software stack for ROS 2.0.

@mikaelarguedas
Copy link
Member

Thanks @sagniknitr for your interest in this.

We didn't have the need for the rest of the utilities so far.
We will happily review any PR that port additional utilities.

@chapulina
Copy link
Contributor

See #58 for fill image

@jacobperron
Copy link
Member

It appears all of the utilities have been migrated. I've opened a PR updating the references mentioned in the description (#72). Regarding the reference to CameraInfo details (http://www.ros.org/wiki/image_pipeline/CameraInfo), it's currently referencing the original documentation under image_pipeline. Perhaps we should consider moving that documentation to this repository to live closer to the message definition. In any case, I think it is out of scope for this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

7 participants