Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
MNT: use stdlib tools in allow_rasterization #6232
Conversation
tacaswell
added the
needs_review
label
Mar 27, 2016
tacaswell
added this to the
2.1 (next point release)
milestone
Mar 27, 2016
|
This looks good to me. Are there any back-compat concerns removing those methods? We could still keep them around in case other projects are using them. |
|
Those functions are local to the decorator def. |
|
Ah, right! Ok. Merging. |
WeatherGod
merged commit eac6ce1
into matplotlib:master
Mar 28, 2016
mdboom
removed the
needs_review
label
Mar 28, 2016
tacaswell
deleted the
tacaswell:mnt_raster_context branch
Mar 28, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
tacaswell commentedMar 27, 2016
Use a context manager to mange enabling/disabling the rasterization.
This will now try to un-wrap the rasterization context state if an
exception occurs.
Use functools.wraps instead of doing it 'manually'.