-
Notifications
You must be signed in to change notification settings - Fork 120
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
Discrepancy between sensor_msgs/CompressedImage docs and format field values published by compressed_image_transport #56
Comments
Determined in code by those lines (for jpeg):
|
The changes date back to 2012 so it is probably documentation that is not up to date. |
The documentation should probably say something like:
But it gets more complex due to depth encoding... |
.. depth encoding reuses the same message
and here So for depth it is like:
Looking into the code empty CODEC is interpreted as png for backwards compatibility |
Fix provided in ros/common_msgs#184 . Feel free to leave some comments there. And thanks for the research you did! |
sensor_msgs/CompressedImage
docs says that acceptableformat
field values are:jpeg
andpng
.compressed_publisher.cpp
produces more verbose format field. E.g.:Is this a bug in code, or bug in docs?
The text was updated successfully, but these errors were encountered: