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.

Sign up for GitHub

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

Jump to bottom

support rosbag play --loop in rospy.Timer #2257

Open
wants to merge 1 commit into
base: noetic-devel
Choose a base branch
from
Open

support rosbag play --loop in rospy.Timer #2257

wants to merge 1 commit into from

Commits on Jul 6, 2022

  1. support rosbag play --loop in rospy.Timer 

    I fail to understand why this issue still exists after all these years.
Without the patch the Timer thread just dies on rosbag loop *with no way of detecting and restarting it* easily.
Instead the patch changes the behavior to assume it slept enough and calls the callback.
If application logic requires resets, this can be detected from the TimerEvent passed to the callback.
    @v4hn
    v4hn authored Jul 6, 2022
    Configuration menu
    Copy the full SHA
    b054b45 View commit details
    Browse the repository at this point in the history