-
Notifications
You must be signed in to change notification settings - Fork 371
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
Ros.js cannot handle very large messages #36
Comments
Web Worker Migrated from #34 @rctoris :
|
The link to original example https://github.com/downloads/RobotWebTools/rosjs/example.tar.gz. It doesn't have a data bag with :( I'll have to generate a large point cloud to reproduce. |
Thought I'd add further insight on this issue. It appears that the ROS.Topic.subscribe code does not implement queue_size on the client side, so data piles up if the listener function can't process data as quickly as it comes it. I implemented a workaround to provide a queue size of 1 by using a separate display "thread" to process the data, leaving the listener.subscribe function to just copy the latest message to a globally shared variable and exit. The display thread is set up using the setInterval() call. I don't know if I should report the queue_size problem as a separate issue or not, but the fact that queue_size is a parameter to ROSLIB.Topic but not used makes it seem like a good candidate. |
related: RobotWebTools/rosbridge_suite#190 |
Migrated from https://github.com/RobotWebTools/rosjs/issues/27
@baalexander :
The text was updated successfully, but these errors were encountered: