ros-perception / image_pipeline

An image processing pipeline for ROS.
Other
786 stars 726 forks source link

getTopicQosProfile not reliable #962

Open Flova opened 6 months ago

Flova commented 6 months ago

In ros iron the QOS policy of the image proc nodes is currently determined by getTopicQosProfile with a default to the default sensor data qos policy. This is an issue, because during launch the incoming topic might not be available yet, so the sensor data qos policy is used. This causes downstream issues, because this qos policy is not compatible with reliable subscribers. So even if the input data has a reliable qos the output data might not in a non-deterministic way.

Flova commented 6 months ago

Also overwrites don't seem to be honored.

mikeferguson commented 6 months ago

These issues should be addressed by https://github.com/ros-perception/image_pipeline/issues/847 - although only targeted at J-turtle currently. Backporting to Iron is going to be quite an effort since rolling/j-turtle have diverged so far from Iron at this point

mikeferguson commented 1 month ago

Finally digging into this again (as part of 847) - and after examining the API interfaces we actually have available in image_transport - here's the solution I'm working on: