On Thu, Apr 23, 2009 at 3:06 PM, Jae-Joon Lee <lee.j.j...@gmail.com> wrote:
> On Tue, Apr 21, 2009 at 10:42 PM, Eric Bruning <eric.brun...@gmail.com> wrote:
>> On a somewhat related note, how are you turning rasterization on and
>> off? Are you using my per-artist patch (which last I knew wasn't in
>> trunk) or some other solution?
>
> I remember that I tried to use your patch, but all the links that I
> found were broken. So I wrote a few lines for monkey patching. It was
> straight forward as I only needed a rasterization of the QuadMesh
> class.

Sorry about the broken links. I've attached a diff made against trunk
from a few days ago.

The discussion about what to do with my patch fizzled. I created a
decorator that made mixed-mode switching a one-line change per artist
type. I also added get/set_rasterized and an _rasterized attribute to
the Artist base class. I've used it on and off for a few months now
with no noted bugs.

If we don't like the decorator, we can just make a helper function
that is called at the beginning of every artist.draw() method. It's
not a very complicated modification.


>
> Are you planning to commit your patch to the trunk? I'll be glad to
> help you if there are any issues.


I'd love to get the patch in trunk, if only so that more people can
try it out and find things to improve (or re-implement).

Thanks,
Eric

Attachment: mixed-mode.diff
Description: Binary data

------------------------------------------------------------------------------
Crystal Reports &#45; New Free Runtime and 30 Day Trial
Check out the new simplified licensign option that enables unlimited
royalty&#45;free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to