matplotlib-devel Mailing List for matplotlib

Ian Thomas wrote:
> Hello all,
> 
> I think I have fixed two bugs in the contouring code (src/cntr.c):
> 1) inconsistent behaviour in how contour and contourf handle saddle
> grid squares, and
> 2) incorrect handling of masked regions in filled contour plots.
> 
> Attached is a gzipped tar file containing an explanation of the bugs
> and what I've done, as well as an svn diff against HEAD and small
> example scripts to demonstrate the bugs before and after the fixes.
> 
> I have tested the fixes for the cases I normally come up with, but
> this is by no means exhaustive.
> 
> Ideally it would be good if someone who is more familiar with cntr.c
> than I am could take a look at what I've done and see if it is OK.
Ian,
I will take a look, and commit the changes if things look OK. I suspect 
Mike D. and I are the most familiar with this code (apart from the 
original author--and I don't know who that is); both of us have tried 
unsuccessfully to solve the masked region problem. If you have solved 
these problems, that's fantastic!
Eric

Hello all,
I think I have fixed two bugs in the contouring code (src/cntr.c):
1) inconsistent behaviour in how contour and contourf handle saddle
grid squares, and
2) incorrect handling of masked regions in filled contour plots.
Attached is a gzipped tar file containing an explanation of the bugs
and what I've done, as well as an svn diff against HEAD and small
example scripts to demonstrate the bugs before and after the fixes.
I have tested the fixes for the cases I normally come up with, but
this is by no means exhaustive.
Ideally it would be good if someone who is more familiar with cntr.c
than I am could take a look at what I've done and see if it is OK.
Ian Thomas
ian...@go...