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

fix coalesce_interval_ms not working correctly #296

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

Conversation

sathyas2023
Copy link

#284

The coalesce_interval_ms parameter doesn't get used properly. Spamming axis events (ex: spinning joysticks) causes rates to get very high.

should_publish is being incorrectly overwritten to true in the event loop function.

The PR makes the event loop logic the same as game_controller_node which did not have this issue from my testing.

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.

1 participant