image_transport_plugins
image_transport_plugins copied to clipboard
Fix compressed subscribe impl
fixed an issue with not implemented subscripeImpl with 5 arguments (options argument)
Hi @PhilippPolterauer! Thanks for the contribution. Unfortunately I run into this error on galactic, foxy, and rolling.
/catkin_ws/src/image_transport_plugins/compressed_image_transport/include/compressed_image_transport/compressed_subscriber.h:65:8: error: ‘void compressed_image_transport::CompressedSubscriber::subscribeImpl(rclcpp::Node*, const string&, const Callback&, rmw_qos_profile_t, rclcpp::SubscriptionOptions)’ marked ‘override’, but does not override
65 | void subscribeImpl(
| ^~~~~~~~~~~~~
In file included from /catkin_ws/src/image_transport_plugins/compressed_image_transport/src/compressed_subscriber.cpp:35:
/catkin_ws/src/image_transport_plugins/compressed_image_transport/include/compressed_image_transport/compressed_subscriber.h:65:8: error: ‘void compressed_image_transport::CompressedSubscriber::subscribeImpl(rclcpp::Node*, const string&, const Callback&, rmw_qos_profile_t, rclcpp::SubscriptionOptions)’ marked ‘override’, but does not override
65 | void subscribeImpl(
| ^~~~~~~~~~~~~
/catkin_ws/src/image_transport_plugins/compressed_image_transport/src/compressed_subscriber.cpp: In member function ‘void compressed_image_transport::CompressedSubscriber::subscribeImpl(rclcpp::Node*, const string&, const Callback&, rmw_qos_profile_t, rclcpp::SubscriptionOptions)’:
/catkin_ws/src/image_transport_plugins/compressed_image_transport/src/compressed_subscriber.cpp:102:11: error: ‘subscribeImplWithOptions’ is not a member of ‘Base’ {aka ‘image_transport::SimpleSubscriberPlugin<sensor_msgs::msg::CompressedImage_<std::allocator<void> > >’}
102 | Base::subscribeImplWithOptions(node, base_topic, callback, custom_qos, options);
Could you kindly take another look at the build?
I'm going to look at it, but it seems i compiled against a wrong version of image_common. which is only available for galactic. maybe you could set up a galactic branch, where this code could fit.
br