|
From: Ethan A M. <merritt@u.washington.edu> - 2006-08-25 16:04:14
|
On Friday 25 August 2006 12:15 am, Petr Mikulik wrote: > > It seems clear to me that at least for gridded surfaces the > > "continue" statement should be replaced by a jump to > > come_here_if_undefined. That does, in fact, fix the specific > > problem reported on the newsgroup. > > > > However, I am dubious about the effect this has on PM3D output. > With your patch of plot3d.c, all scans have the number of points as > intended. The rectangle(s) contaning a point with NaN are not drawn. This > looks like to be what is expected, so I propose you commit the plot3d.c > patch. I think there is still a problem with the patched version. I am not sure where it is, though. Have a look at the attached PNG image compared before + after the patch. The input data file is "triangle.dat" with two individual values replaced by "Junk" and "NaN" respectively. The patched version gets it almost right, but only in the case of "splot ... using ($1):($2):($3)" Even so, instead of skipping a single rectangle, it omits the entire rest of the scan line. Any ideas where it could be going wrong? -- Ethan A Merritt Biomolecular Structure Center University of Washington, Seattle 98195-7742 |