Conversation
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
|
@mjcarroll some ideas why this might be usefull Lower callback latency: callbacks are dispatched immediately instead of waiting up to one 30 Hz tick (~33 ms). This is critical for time-sensitive data such as TF transforms and sensor messages. ║ Update loop never blocked: the Qt render/update loop runs at its full rate without being delayed by executor work. This keeps the UI responsive under load. Higher throughput: all N messages are processed faster because the executor is not rate-limited to 30 Hz ticks. |
|
I made in this branch a benchmark test comparing both solutions. This is the result
|
asymingt
left a comment
There was a problem hiding this comment.
I like this change. Are you certain that it won't introduce any data races on variables accessed from within executor callbacks, now that they are accessed from a separate thread?
No description provided.