ros-perception / image_common

Common code for working with images in ROS
http://www.ros.org/wiki/image_common
123 stars 219 forks source link

[ROS2 Foxy] `image_transport_plugins` broken #184

Open Myzhar opened 3 years ago

Myzhar commented 3 years ago

There are two problems with image_transport_plugins that were not present in ROS2 Eloquent and they are both related to compressed image publisher.

The first time create_camera_publisher is called this error is thrown: [ERROR] Failed to load plugin image_transport/compressed_pub, error string: parameter 'format' could not be set: Not supported

When you call create_camera_publisher to instantiate other publishers: [ERROR] Failed to load plugin image_transport/compressed_pub, error string: parameter 'format' has already been declared

It seems that there is something wrong with the node namespace and the parameters namespace...

You can read more here: https://github.com/stereolabs/zed-ros2-wrapper/issues/31

As test code you can look here: https://github.com/stereolabs/zed-ros2-wrapper/blob/dev_foxy/zed_components/src/zed_camera/src/zed_camera_component.cpp#L1656-L1695

laszloturanyi commented 3 years ago

This has been merged. https://github.com/ros-perception/image_transport_plugins/pull/58 I tried to compile image_transport_plugins from source and it worked fine.

Void12987 commented 2 years ago

I can confirm ZED2 starts on Foxy Docker (specifically sudo docker run --gpus all -it --privileged dustynv/ros:foxy-slam-l4t-r32.6.1) but image_transport won't load so the image/depth data can't be echoed.

I can also confirm that Bmoradi93's comment fixes getting ZED data but at the same time the ERROR message that image_transport failed to load persists.

ruffsl commented 2 years ago

dustynv/ros:foxy-slam-l4t-r32.6.1

LAST PUSHED 6 months ago by dustynv

Has this image been updated with respect to the latest foxy sync?