FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

FigureCanvasMac.flush_events() raises NotImplementedError · Issue #197 · matplotlib/matplotlib · GitHub

Repository navigation

FigureCanvasMac.flush_events() raises NotImplementedError #197

Closed

Description

Original report at SourceForge, opened Thu Mar 10 10:39:14 2011

FigureCanvasMac seems like it should be a "backend with GUI", but it doesn't implement flush_events(). Seems like it should pass, if nothing else.

SourceForge History

  • On Fri Mar 25 18:54:33 2011, by pivanov314: assigned_to: 100

Activity

  1. dmcdougall commented on Sep 25, 2012

    Member

    I'm not sure there's a Cocoa API for this:

    http://www.cocoabuilder.com/archive/cocoa/149352-flushing-events.html
    http://cocoadev.com/wiki/FlushAppEventQueue

    It appears that event polling was phased out after OS 9. Perhaps a pass is the best option here.

  2. mdehoon commented on Dec 12, 2012

    Contributor

    Actually why is a flush_events method needed? It is not used in the matplotlib source code. Perhaps it is better to remove flush_events altogether?

  3. efiring commented on Jul 22, 2013

    Member

    @mdehoon, flush_events dates back to February, 2008 as part of the official API for gui backends, so even though we don't use it internally or in examples, we can't just rip it out. It appears to be carefully implemented in the other gui backends, and at least for them it seems to be a reasonable method to have available. Therefore I think that adding it to the macosx backend as a no-op (if indeed there is not equivalent functionality available) would be the simplest way to solve the problem and close this issue.

  4. mdehoon commented on Jul 22, 2013

    Contributor

    OK, I'll add it then. This can be done on MacOSX using NSView displayIfNeeded.
    If somebody could please accept pull request #2233 , then I'll create a new pull request for flush_events.

  5. efiring commented on Jul 23, 2013

    Member

    On 2013/07/22 1:49 PM, mdehoon wrote:

    If somebody could please accept pull request #2233
    #2233 , then I'll
    create a new pull request for flush_events.

    Done.

  6. mdehoon commented on Jul 23, 2013

    Contributor

    @efiring Thanks. Pull request #2245 adds a flush_events method to the MacOSX backend.

  7. dmcdougall commented on Aug 18, 2013

    Member

    Resolved by #2245.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL