ros-noetic icon indicating copy to clipboard operation
ros-noetic copied to clipboard

misc: bump image_transport_plugins to 1.15.0-1

Open daenny opened this issue 1 year ago • 6 comments

I am not completely sure if this is the correct way, but https://github.com/ros/rosdistro/pull/42174 was merged to the ros-distro. So I would like bump image transport plugins on robostack-staging to 1.15.0.

daenny avatar Aug 26 '24 11:08 daenny

Bump, do I need to do anything else to get these published?

daenny avatar Sep 11 '24 07:09 daenny

Sorry I’ve been really busy with teaching and paper/grant writing.

I think we shouldn’t bump individual packages for reasons outlined in https://github.com/RoboStack/ros-humble/pull/204

We should instead aim for a full rebuild of all packages.

Tobias-Fischer avatar Sep 11 '24 07:09 Tobias-Fischer

Okay, is there any process to do that? The problem at the moment is, that the image_transport_plugins is broken on ros-distro-mutex 0.5 due to an opencv update.

daenny avatar Sep 11 '24 08:09 daenny

Okay, is there any process to do that? The problem at the moment is, that the image_transport_plugins is broken on ros-distro-mutex 0.5 due to an opencv update.

In which sense? As a temporary solution have you considered pinning opencv to a working version?

traversaro avatar Sep 11 '24 08:09 traversaro

https://github.com/ros/rosdistro/issues/42149

And: https://github.com/ros-perception/image_transport_plugins/pull/130

ros-distro-mutex 0.5 pins opencv to 4.9. This introduced strict assert check on params.size() in various functions. There is AFAIK no other way releasing the correct version of image_transport_plugins as opencv is pinned by the mutex. Currently we have released the patched version on an internal quetz server.

daenny avatar Sep 11 '24 09:09 daenny

Thanks, now the situation is clear to me.

traversaro avatar Sep 11 '24 11:09 traversaro

Will be done in #501

Tobias-Fischer avatar Jan 03 '25 06:01 Tobias-Fischer