Skip to content
This repository has been archived by the owner on Jan 7, 2023. It is now read-only.

Use default sensor qos for sensor topics #107

Open
wants to merge 1 commit into
base: eloquent
Choose a base branch
from

Conversation

ruffsl
Copy link

@ruffsl ruffsl commented Feb 5, 2020

Sensor data
For sensor data, in most cases it’s more important to receive readings in a timely fashion, rather than ensuring that all of them arrive. That is, developers want the latest samples as soon as they are captured, at the expense of maybe losing some. For that reason the sensor data profile uses best effort reliability and a smaller queue depth.
https://index.ros.org/doc/ros2/Concepts/About-Quality-of-Service-Settings/#qos-profiles

References:

@ruffsl
Copy link
Author

ruffsl commented Feb 21, 2020

This could effect the rosbag recording, given rosbag does not of yet auto discover QoS setting:
ros2/rosbag2#125

@emersonknapp
Copy link

Rosbag2 can now correctly record SensorData QoS profiles as of ros2/rosbag2#355 - this change will be in the Foxy release

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants