On Thu, Dec 4, 2008 at 11:44 AM, Scott Webster <[EMAIL PROTECTED]> wrote:
> Thanks! Doesn't crash anymore. Unfortunately, I think there is still
> some problem.
>
> I didn't immediately find where the box drawing code was, doesn't
> appear to be in libprocess/stats.c, but this does only seem to happen
> with the median option so maybe the problem is there... Maybe I'll
> try to understand the code if I have time, but it is somewhat
> overwhelming at first. I figured I might as well report this now.
>
I think I found/fixed the problem. A missing "+col" in the code. See
the diff between the Dec. 2nd nightly and my code below (includes
David's and my change). Seems to work now. By the way, should I post
bug-like stuff such as this to the -users or -devel list? Certainly
this seems to have become a more -devel related topic with all the
code flying around.
Scott
--- gwyddion-2.10.20081202/libprocess/stats.c 2008-12-01
18:58:42.000000000 -0800
+++ gwyddion-custom/libprocess/stats.c 2008-12-04 14:56:50.897096040 -0800
@@ -3957,8 +3957,8 @@
buf = gwy_data_line_new(height, 1.0, FALSE);
for (j = 0; j < width; j++) {
gwy_data_field_get_column_part(data_field, buf,
- j, row, row + height);
- ldata[j] = gwy_math_median(width, buf->data);
+ j+col, row, row + height);
+ ldata[j] = gwy_math_median(height, buf->data);
}
g_object_unref(buf);
break;
------------------------------------------------------------------------------
SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada.
The future of the web can't happen without you. Join us at MIX09 to help
pave the way to the Next Web now. Learn more and register at
http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/
_______________________________________________
Gwyddion-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gwyddion-users
