Skip to content
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

BUG : restore allow_rasterization on PolyCollection #3306

Merged
merged 1 commit into from
Jul 25, 2014

Conversation

tacaswell
Copy link
Member

The was accidentally removed as part of
b8726d0

Closes #3299

@mdboom can you take a look at this?

The was accidentally removed as part of
b8726d0

Closes matplotlib#3299
@tacaswell tacaswell added this to the v1.4.0 milestone Jul 25, 2014
@mdboom
Copy link
Member

mdboom commented Jul 25, 2014

Looks good.

mdboom added a commit that referenced this pull request Jul 25, 2014
BUG : restore allow_rasterization on PolyCollection
@mdboom mdboom merged commit 8a8d40e into matplotlib:v1.4.x Jul 25, 2014
@tacaswell tacaswell deleted the polycollection_allow_raster branch July 25, 2014 23:31
@letmaik
Copy link
Contributor

letmaik commented Jul 26, 2014

Quick question, this means that for _CollectionWithSizes, EllipseCollection, PathCollection, PolyCollection, RegularPolyCollection, and CircleCollection there are effectively two nested invocations of the allow_rasterization decorator. Is that how it should be?
For Collection, PatchCollection, TriMesh, QuadMesh this is not the case.

@tacaswell
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants