director icon indicating copy to clipboard operation
director copied to clipboard

Director cannot handle collections messages at very high rate

Open mauricefallon opened this issue 9 years ago • 2 comments

Not going to fix it now, reporting anyway:

Director locks up when receiving collection messages (from pronto in debug mode) at more than 300Hz or so. Need to disable collections in startup

mauricefallon avatar Feb 03 '17 18:02 mauricefallon

It's likely because of these lines:

https://github.com/RobotLocomotion/director/blob/master/src/python/director/lcmcollections.py#L207

The way collections was implemented in Director (by following lcmgl.py's implementation) put the lcm subscribers in Python. But as we know, python cannot decode messages at a fast rate. The best fix would be to move the lcm subscribers to threaded c++. If you feel strongly about it I can help.

If you removed the lines calling setNotifyAllMessagesEnabled(True) then it might resolve the lock up issue, but it means some messages will be skipped.

patmarion avatar Feb 03 '17 19:02 patmarion

This is used for analysis - so allowing messages to be skipped is not a suitable option. I think the correct option is to move decoding to c++. I'll come back to this when I have the cycles.

mauricefallon avatar Feb 04 '17 15:02 mauricefallon