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

Improve uorb subscription interval handling #648

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

jlaitine
Copy link

@jlaitine jlaitine commented Feb 22, 2024

This fixes an issue found with "topic_listener -r 1" command;

The topic which is subscribed with a rate of 1 second is initially given twice, and after that every 1 second.

The time when the topic was last updated can't be initialized to 0. The best possible initial value is "now - interval", which gives the subscriber the first publish after subscription, and after that according to the interval.

…tervals

Set the last_update to now-interval, instead of 0.

Signed-off-by: Jukka Laitinen <[email protected]>
@jlaitine jlaitine force-pushed the improve_uorb_subscription_interval_handling branch from 4839dc7 to 66546c4 Compare February 22, 2024 10:14
@jlaitine jlaitine requested a review from pussuw February 22, 2024 10:15
@jlaitine jlaitine merged commit 415da09 into main Feb 23, 2024
24 of 25 checks passed
@jlaitine jlaitine deleted the improve_uorb_subscription_interval_handling branch February 23, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants