-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Better throttling #630
Comments
for reference for others reading this thread: Lodash implementation of _.throttle: |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Just learned about Perhaps there's something that can be done with it? |
As pointed out here: https://groups.google.com/forum/m/#!topic/dc-js-user-group/3AlQht_8Eqs
The current brush throttling is actually a debounce: it prevents more than one brushing event from happening in a time interval, but it also prevents any events from happening while there are events coming in at a higher rate, and then just fires the last one at the end.
What we really want is to fire redraws as fast as we can without them backing up. So we start the first redraw, then throw out all but the last event we get while that's happening, then start the next redraw, etc.
Also see #627: some would prefer having no redraws at all until the end of brushing.
The text was updated successfully, but these errors were encountered: