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

Timer callback stops firing when no data is moving through the system #158

Closed
AlexDayCRL opened this issue Apr 17, 2024 · 6 comments
Closed
Assignees

Comments

@AlexDayCRL
Copy link
Contributor

Hi,

I am having troubles with my timer callback calling after a few seconds when using the zenoh middleware with my application. I am not experiencing this issue with cyclonedds. I have tried and failed to reproduce it with a small example. Is there any way to introspect the timer queue within Zenoh?

@clalancette
Copy link
Collaborator

So it is tricky because rmw_zenoh doesn't directly deal with the timer callbacks. Those are handled at the rcl/rclcpp layer. However, if the rmw_wait that is implemented in Zenoh gets hung up for some reason, it can cause the timer callbacks to be delayed. They should eventually fire (after rmw_wait times out), but they may be delayed.

It's hard to say here without an example, but there have been some other issues we've been trying to track down with rmw_wait. #153 should have solved some of it, but seems to have caused a regression. If you apply #157, does the problem go away?

@AlexDayCRL
Copy link
Contributor Author

@clalancette I still have the same issues on #157. From your comments it seems that I would want to instrument the rmw_wait function if I wanted to drill down into what was happening though.

@MichaelOrlov
Copy link

Moving to the backlog for now since other relevant issues #153 and #157 haven't been resolved yet.

@AlexDayCRL
Copy link
Contributor Author

This issue also only happens when my node has multiple timers.

@Yadunund
Copy link
Member

Hey @AlexDayCRL could you try doing a clean build with the latest rolling branch and confirm if the problem still exists?

@clalancette
Copy link
Collaborator

There's been no response here in a long time, so I'm going to close this out. If you are still having problems with the latest, please feel free to reopen.

@clalancette clalancette closed this as not planned Won't fix, can't repro, duplicate, stale Aug 21, 2024
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

No branches or pull requests

4 participants