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

Regression in maximum path size · Issue #4407 · matplotlib/matplotlib · GitHub

Repository navigation

Regression in maximum path size #4407

Closed

Description

With matplotlib 1.4.3 it is possible to draw at least a random time series of 70,000 points on the Qt4Agg backend:

$ python -c 'import numpy as np; from matplotlib import pyplot as plt; plt.plot(np.random.rand(70000)); plt.show()'

With the current HEAD, an OverflowError is thrown:

OverflowError: In draw_path: Exceeded cell block limit

Perhaps it's just a transient regression, but I didn't see it mentioned anywhere.

Activity

  1. WeatherGod commented on May 5, 2015

    Member

    This sounds very familiar, but I could have sworn it was something to do
    with the macosx backend and old limitations of quartz.

    On Tue, May 5, 2015 at 1:58 PM, Antony Lee notifications@github.com wrote:

    With matplotlib 1.4.3 it is possible to draw at least a random time series
    of 70,000 points on the Qt4Agg backend:

    $ python -c 'import numpy as np; from matplotlib import pyplot as plt; plt.plot(np.random.rand(70000)); plt.show()'

    With the current HEAD, an OverflowError is thrown:

    OverflowError: In draw_path: Exceeded cell block limit

    Perhaps it's just a transient regression, but I didn't see it mentioned
    anywhere.

    —
    Reply to this email directly or view it on GitHub
    #4407.

  2. efiring commented on May 5, 2015

    Member

    It seems odd that 1.4.3 and master are behaving differently; I think that with path simplification, both should be able to handle the test case.

  3. mdboom commented on May 5, 2015

    Member

    Do you have any rcParams in effect (in your matplotlibrc for example)?

  4. mdboom commented on May 5, 2015

    Member

    This may have to do with "upgrading" to Agg-svn from Agg-2.4.

  5. anntzer commented on May 5, 2015

    ContributorAuthor

    I thought both would pick up the same matplotlibrc anyways but just to be sure I removed it, and see the same difference.

  6. self-assigned this
    on May 5, 2015
  7. mdboom commented on May 5, 2015

    Member

    See #4408.

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

Metadata

Metadata

Assignees

Labels

No labels
No 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