Port parameter declared check compressed image transport subscription

[p0wdrdotcom]

This fixes a bug where more than one compressed image subscription would fail due to the following exception.

terminate called after throwing an instance of 'rclcpp::exceptions::ParameterAlreadyDeclaredException' what(): parameter 'mode' has already been declared
This has been fixed in future versions (galactic) but is not available for foxy.

[wep21]

This resolves #83.

[wep21]

@mjcarroll
ros2-gbp/image_transport_plugins-release#2
Does galactic release refer to foxy-devel?

[wep21]

@mjcarroll @jacobperron friendly ping

[wep21]

@mjcarroll @jacobperron friendly ping

[jacobperron]

For posterity, this is a partial backport of #73.

Perhaps we should include similar changes to the publisher code as well to avoid the same exception. What do you think @p0wdrdotcom @wep21 ?

[wep21]

@jacobperron I created a PR including modification for the publisher and the subscriber derived from #73 in #86.
Could you review the change?

[wep21]

@jacobperron friendly ping

[ijnek]

I'd like this issue fixed but there seems to be not much progress made here.

Who's the maintainer of this package? @dgossow is listed in package.xml, but is that up-to-date?

[jacobperron]

I think @mjcarroll might be the maintainer (though I can also help out).

I think this was fixed in #86. It hasn't been released yet though. I can make a release now. Let me know the issue persists.

[jacobperron]

Releases for Foxy and Galactic:

• image_transport_plugins: 2.3.3-1 in 'foxy/distribution.yaml' [bloom] ros/rosdistro#32718
• image_transport_plugins: 2.3.3-1 in 'galactic/distribution.yaml' [bloom] ros/rosdistro#32719

@mjcarroll FYI, I bumped the minor for Rolling to make room for patch releases for Foxy/Galactic.

[ijnek]

Great, thanks @jacobperron