Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

contributes an out-of-the-box durability profile #111

Open
wants to merge 2 commits into
base: rolling
Choose a base branch
from

Conversation

BrannonKing
Copy link

Things like a map server benefit from a durable message (aka, send the last published map to each subscriber as they subscribe). We should have one of these in the default profile list, especially with talk of being able to select from canned QoS profiles using command line parameters.

@BrannonKing
Copy link
Author

I assume that rclpy defines these as well and we would need a change there too?

Also, in thinking about it further I want depth 1 instead of depth 2. As this is typically used for large messages that change very infrequently, it is unlikely to benefit from a "finish sending that last one before this new one goes out" scenario.

@mikaelarguedas mikaelarguedas added the in review Waiting for review (Kanban column) label Jul 19, 2017
@mikaelarguedas
Copy link
Member

thanks @BrannonKing for the contribution. We will delay this until we iterated on how to do the equivalent of ROS 1 "latching" in ROS2.

@mikaelarguedas mikaelarguedas added ready Work is about to start (Kanban column) and removed in review Waiting for review (Kanban column) labels Jul 27, 2017
@mikaelarguedas mikaelarguedas added enhancement New feature or request and removed ready Work is about to start (Kanban column) labels Aug 3, 2017
@wjwwood
Copy link
Member

wjwwood commented Feb 22, 2018

Here's the issue we're using to track the latching decision: ros2/ros2#464

@paulbovbel
Copy link

paulbovbel commented May 5, 2018

Also, in thinking about it further I want depth 1 instead of depth 2

+1, this definitely makes sense for a 'default' durability profile, if approximating 'latched' behaviour from ROS1

@audrow audrow changed the base branch from master to rolling June 28, 2022 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants