From: Eric F. <ef...@ha...> - 2010-01-15 17:59:39
|
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 |