Fix #2249: Topic statistics do not handle well ROS time jumping back#2250
Fix #2249: Topic statistics do not handle well ROS time jumping back#2250peci1 wants to merge 2 commits intoros:noetic-develfrom
Conversation
…g topic statistics.
ef5cbaa to
025980c
Compare
|
And the fix comes. |
|
Is there actually reason to copy the whole stats structure twice in each callback? Wouldn't it be better to modify it directly via an |
|
@mjcarroll @sloretz friendly ping |
|
Thank you for the PR! ROS Noetic will reach end-of-life on May 31st, 2025. Every change comes with a risk of introducing regressions, and there isn't much time left to fix them. To make sure this PR doesn't introduce any regressions please:
|
|
This PR comes with a unit test that fails without this PR and succeeds with it. The test does something that should be completely valid - having topic statistics enabled, receiving lots of messages and then resetting ROS time back. |
|
ROS 1 is end-of-life (EOL) as of today, May 31st 2025. I am archiving this repository because:
If you still rely on ROS 1, read this page to learn about your options. |
First, I'm submitting a failing test.
It makes 4 repetitions of receiving 10000 messages, each time resetting ROS time from 1.0 onwards. First repetition is okay. If #2249 is not fixed, the following iterations will take considerably longer because the internal lists are never cleared.