Thanks to visit codestin.com
Credit goes to github.com

Skip to content

idle_event is invoked only once #959

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

Closed
bgamari opened this issue Jun 19, 2012 · 6 comments
Closed

idle_event is invoked only once #959

bgamari opened this issue Jun 19, 2012 · 6 comments
Assignees
Labels
Milestone

Comments

@bgamari
Copy link
Contributor

bgamari commented Jun 19, 2012

FigureCanvasBase.idle_event doesn't return anything, meaning it is only called once under the gtk backend. This leads to the apparent breakage of which means examples/idle_and_timeout.py is broken.

All in all it seems that the idle_event callback is incompletely implemented at best. It's quite unclear whether it's even a good idea to pipe this event through CallbackRegistry, since we only really want the backend to register it (with, e.g., gobject.idle_add) if there is a callback to be called. Otherwise we are just creating unnecessary work by executing a stub function.

@efiring
Copy link
Member

efiring commented Jul 22, 2013

It looks like so little of the idle_event machinery is implemented that it might be best to rip out what is there, none of which seems to be functional in any way.

@mdboom
Copy link
Member

mdboom commented Jul 22, 2013

It's good to know this is broken -- I have big plans for it. There was discussion at Scipy of having a "draft mode" that would switch on during panning and zooming (to make it faster) and then have it do a high-quality rendering on idle once the panning stops. So there's a good use case for it -- but clearly it's broken. I'm happy to look into why as part of the above work when I get to it.

@tacaswell
Copy link
Member

@mdboom What is the state of this?

@tacaswell tacaswell added this to the v2.0 milestone Aug 18, 2014
@tacaswell tacaswell modified the milestones: v2.0, v1.5.x Aug 18, 2014
@tacaswell
Copy link
Member

Closing as this is fixed in #3769 as noted by @ahaldane.

Thanks for noting this and closing such a long-standing bug!

@story645
Copy link
Member

The docs say idle_event is deprecated (http://matplotlib.org/examples/event_handling/idle_and_timeout.html), so is the issue this is referring to still exist?

@tacaswell
Copy link
Member

Closing as we have fully deprecated idle_event as of #8454.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants