X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fmath%2Ftrimmed-mean.c;h=a687dd9b5fcffcd711ed983d0566edbb54847213;hb=feba48309a227fe40feb3a87cbe900015021ac73;hp=d1cc6b708279098ab8d263b95923d9b31c968ed6;hpb=8b71948cd57dbd2787cb4c50525b957e9be8a62b;p=pspp diff --git a/src/math/trimmed-mean.c b/src/math/trimmed-mean.c index d1cc6b7082..a687dd9b5f 100644 --- a/src/math/trimmed-mean.c +++ b/src/math/trimmed-mean.c @@ -1,5 +1,5 @@ /* PSPP - a program for statistical analysis. - Copyright (C) 2008, 2009 Free Software Foundation, Inc. + Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -15,15 +15,17 @@ along with this program. If not, see . */ #include -#include "trimmed-mean.h" -#include -#include -#include -#include +#include "math/trimmed-mean.h" + #include -#include +#include "data/val-type.h" +#include "libpspp/assertion.h" +#include "libpspp/cast.h" +#include "math/order-stats.h" + +#include "gl/xalloc.h" static void acc (struct statistic *s, const struct ccase *cx UNUSED, double c, double cc, double y) @@ -31,11 +33,11 @@ acc (struct statistic *s, const struct ccase *cx UNUSED, double c, double cc, do struct trimmed_mean *tm = UP_CAST (s, struct trimmed_mean, parent.parent); struct order_stats *os = &tm->parent; - if ( cc > os->k[0].tc && cc < os->k[1].tc) - tm->sum += c * y; + if ( cc > os->k[0].tc && cc <= os->k[1].tc) + tm->sum += c * y; - if ( tm->cyk1p1 == SYSMIS && cc >os->k[0].tc) - tm->cyk1p1 = c * y; + if ( tm->cyk1p1 == SYSMIS && cc > os->k[0].tc) + tm->cyk1p1 = c * y; } static void @@ -55,7 +57,7 @@ trimmed_mean_create (double W, double tail) struct statistic *stat = &os->parent; os->n_k = 2; - os->k = xcalloc (sizeof (*os->k), 2); + os->k = xcalloc (2, sizeof (*os->k)); assert (tail >= 0); assert (tail <= 1); @@ -79,18 +81,13 @@ trimmed_mean_calculate (const struct trimmed_mean *tm) { const struct order_stats *os = (const struct order_stats *) tm; - assert (os->cc == tm->w); - return ( - (os->k[0].cc_p1 - os->k[0].tc) * os->k[0].y_p1 - - - (os->k[1].cc - os->k[1].tc) * os->k[1].y_p1 + (os->k[0].cc - os->k[0].tc) * os->k[0].y_p1 + + + (tm->w - os->k[1].cc - os->k[0].tc) * os->k[1].y_p1 + - tm->sum - - - tm->cyk1p1 + tm->sum ) - / - ( (1.0 - 2 * tm->tail) * tm->w); + / ((1.0 - tm->tail * 2) * tm->w); }