Re: [matplotlib-devel] bug in path simplification

2008-08-11 Thread Michael Droettboom
Simplification is now turned off whenever there are nonfinite elements in the vertices array. The "should_simplify" determination is now made in Python (to make it easier to tweak and cache). I also committed your patch to handle masked arrays in the same way as arrays-with-nonfinite values (w

Re: [matplotlib-devel] bug in path simplification

2008-08-11 Thread Michael Droettboom
Michael Droettboom wrote: > So the easy fix is to turn off simplification when the array contains > NaNs (and bonus points if we can cache that so we don't have to run > through the list to find NaNs ahead of time). > On further thought, this shouldn't be too difficult -- so I'll go ahead and

Re: [matplotlib-devel] bug in path simplification

2008-08-11 Thread Michael Droettboom
The simplification code was written with the assumption that all of the codes are LINETO. That is, it has no MOVETOs or CURVEs. There is code in backend_agg.h that tries to make sure not to run simplification when this is the case (see should_simplify -- it returns false whenever there is a c