[Pspp-cvs] pspp/src/math/linreg predict.c linreg.c linreg....

[Jason H Stover]

CVSROOT:        /sources/pspp
Module name:    pspp
Branch:         
Changes by:     Jason H Stover <address@hidden> 06/04/09 19:01:11

Modified files:
        src/math/linreg: predict.c linreg.c linreg.h coefficient.c 

Log message:
        added residual function to linreg struct; tidied up

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/pspp/pspp/src/math/linreg/predict.c.diff?tr1=1.3&tr2=1.4&r1=text&r2=text
http://cvs.savannah.gnu.org/viewcvs/pspp/pspp/src/math/linreg/linreg.c.diff?tr1=1.5&tr2=1.6&r1=text&r2=text
http://cvs.savannah.gnu.org/viewcvs/pspp/pspp/src/math/linreg/linreg.h.diff?tr1=1.5&tr2=1.6&r1=text&r2=text
http://cvs.savannah.gnu.org/viewcvs/pspp/pspp/src/math/linreg/coefficient.c.diff?tr1=1.7&tr2=1.8&r1=text&r2=text

Patches:
Index: pspp/src/math/linreg/coefficient.c
diff -u pspp/src/math/linreg/coefficient.c:1.7 
pspp/src/math/linreg/coefficient.c:1.8
--- pspp/src/math/linreg/coefficient.c:1.7      Sun Apr  9 14:45:40 2006
+++ pspp/src/math/linreg/coefficient.c  Sun Apr  9 19:01:11 2006
@@ -1,22 +1,22 @@
 /*
- * lib/linreg/coefficient.c
- * 
- * Copyright (C) 2005 Free Software Foundation, Inc. Written by Jason H Stover.
- * 
- * 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 the Free
- * Software Foundation; either version 2 of the License, or (at your option)
- * any later version.
- * 
- * This program is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
- * more details.
- * 
- * You should have received a copy of the GNU General Public License along with
- * this program; if not, write to the Free Software Foundation, Inc., 51
- * Franklin Street, Fifth Floor, Boston, MA 02111-1307, USA.
- */
+  lib/linreg/coefficient.c
+  
+  Copyright (C) 2005 Free Software Foundation, Inc. Written by Jason H Stover.
+  
+  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 the Free
+  Software Foundation; either version 2 of the License, or (at your option)
+  any later version.
+  
+  This program is distributed in the hope that it will be useful, but WITHOUT
+  ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+  FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+  more details.
+  
+  You should have received a copy of the GNU General Public License along with
+  this program; if not, write to the Free Software Foundation, Inc., 51
+  Franklin Street, Fifth Floor, Boston, MA 02111-1307, USA.
+*/
 
 /*
   Accessor functions for matching coefficients and variables.
@@ -31,14 +31,14 @@
 struct varinfo
 {
   const struct variable *v;    /* Variable associated with this
-                                * coefficient. Note this variable
-                                * may not be unique. In other words,
-                                * a coefficient structure may have
-                                * other v_info's, each with its own
-                                * variable. */
+                                  coefficient. Note this variable
+                                  may not be unique. In other words,
+                                  a coefficient structure may have
+                                  other v_info's, each with its own
+                                  variable. */
   const union value *val;      /* Value of the variable v which this varinfo
-                                * refers to. This member is relevant only to
-                                * categorical variables. */
+                                  refers to. This member is relevant only to
+                                  categorical variables. */
 };
 
 void
@@ -189,8 +189,8 @@
                       const struct variable *v, const union value *val)
 {
   int i = 1;
-  struct pspp_linreg_coeff *result;
-  const struct variable *tmp;
+  struct pspp_linreg_coeff *result = NULL;
+  const struct variable *tmp = NULL;
 
   if (c == NULL)
     {
Index: pspp/src/math/linreg/linreg.c
diff -u pspp/src/math/linreg/linreg.c:1.5 pspp/src/math/linreg/linreg.c:1.6
--- pspp/src/math/linreg/linreg.c:1.5   Sun Apr  9 14:00:04 2006
+++ pspp/src/math/linreg/linreg.c       Sun Apr  9 19:01:11 2006
@@ -115,6 +115,7 @@
    */
   c->method = PSPP_LINREG_SWEEP;
   c->predict = pspp_linreg_predict;
+  c->residual = pspp_linreg_residual;
 
   return c;
 }
Index: pspp/src/math/linreg/linreg.h
diff -u pspp/src/math/linreg/linreg.h:1.5 pspp/src/math/linreg/linreg.h:1.6
--- pspp/src/math/linreg/linreg.h:1.5   Sun Apr  9 14:00:04 2006
+++ pspp/src/math/linreg/linreg.h       Sun Apr  9 19:01:11 2006
@@ -161,6 +161,10 @@
 
   double (*predict) (const struct variable **, const union value **,
                     const void *, int);
+  double (*residual) (const struct variable **,
+                     const union value **,
+                     const union value *,
+                     const void *, int);
 };
 
 typedef struct pspp_linreg_cache_struct pspp_linreg_cache;
@@ -189,6 +193,6 @@
 pspp_linreg_predict (const struct variable **, const union value **,
                     const pspp_linreg_cache *, int);
 double
-pspp_linreg_residual (const struct variable *, const union value **,
+pspp_linreg_residual (const struct variable **, const union value **,
                      const union value *, const pspp_linreg_cache *, int);
 #endif
Index: pspp/src/math/linreg/predict.c
diff -u pspp/src/math/linreg/predict.c:1.3 pspp/src/math/linreg/predict.c:1.4
--- pspp/src/math/linreg/predict.c:1.3  Sat Apr  8 02:47:54 2006
+++ pspp/src/math/linreg/predict.c      Sun Apr  9 19:01:11 2006
@@ -1,26 +1,26 @@
-/* lib/linreg/predict.c
-
- Copyright (C) 2005 Free Software Foundation, Inc.
- Written by Jason H. Stover.
+/*
+   lib/linreg/predict.c
+  
+   Copyright (C) 2005 Free Software Foundation, Inc. Written by Jason H. 
Stover.
 
- 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
- the Free Software Foundation; either version 2 of the License, or (at
- your option) any later version.
-
- This program is distributed in the hope that it will be useful, but
- WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with this program; if not, write to the Free Software
- Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- 02111-1307, USA.
-*/
+   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 the Free
+   Software Foundation; either version 2 of the License, or (at your option)
+   any later version.
+   
+   This program is distributed in the hope that it will be useful, but WITHOUT
+   ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+   FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+   more details.
+   
+   You should have received a copy of the GNU General Public License along with
+   this program; if not, write to the Free Software Foundation, Inc., 51
+   Franklin Street, Fifth Floor, Boston, MA 02111-1307, USA.
+ */
 
 #include <math/linreg/linreg.h>
 #include <math/linreg/coefficient.h>
+#include <gl/xalloc.h>
 
 /*
   Predict the value of the dependent variable with the
@@ -29,15 +29,17 @@
   in the same order.
  */
 double
-pspp_linreg_predict (const struct variable **predictors, 
-                    const union value **vals, 
-                    const pspp_linreg_cache *c,
-                    int n_vals)
+pspp_linreg_predict (const struct variable **predictors,
+                    const union value **vals,
+                    const pspp_linreg_cache * c, int n_vals)
 {
   int i;
+  int j;
+  const struct pspp_linreg_coeff **found;
+  const struct pspp_linreg_coeff *coe;
   double result;
   double tmp;
-  
+
   if (predictors == NULL || vals == NULL || c == NULL)
     {
       return GSL_NAN;
@@ -47,32 +49,44 @@
       /* The stupid model: just guess the mean. */
       return c->depvar_mean;
     }
-  result = c->coeff->estimate; /* Intercept. */
+  found = xnmalloc (c->n_coeffs, sizeof (*found));
+  *found = c->coeff;
+  result = c->coeff->estimate; /* Intercept. */
 
   /*
-    Stop at the minimum of c->n_coeffs and n_vals in case
-    the caller passed us inadequate information, such as too
-    few or too many values.
+    The loops guard against the possibility that the caller passed us
+    inadequate information, such as too few or too many values, or
+    a redundant list of variable names.
    */
-  n_vals++;
-  for (i = 1; i < c->n_coeffs && i < n_vals; i++)
+  for (j = 0; j < n_vals; j++)
     {
-      tmp = pspp_linreg_coeff_get_est (pspp_linreg_get_coeff (c, 
predictors[i], vals[i]));
-      if (predictors[i]->type == NUMERIC)
+      coe = pspp_linreg_get_coeff (c, predictors[j], vals[j]);
+      i = 1;
+      while (found[i] != coe && i < c->n_coeffs)
        {
-         tmp *= vals[i]->f;
+         i++;
+       }
+      if (i < c->n_coeffs)
+       {
+         found[i] = coe;
+         tmp = pspp_linreg_coeff_get_est (coe);
+         if (predictors[j]->type == NUMERIC)
+           {
+             tmp *= vals[j]->f;
+           }
+         result += tmp;
        }
-      result += tmp;
     }
+  free (found);
+
   return result;
 }
 
 double
-pspp_linreg_residual (const struct variable *predictors,
+pspp_linreg_residual (const struct variable **predictors,
                      const union value **vals,
                      const union value *obs,
-                     const pspp_linreg_cache *c,
-                     int n_vals)
+                     const pspp_linreg_cache * c, int n_vals)
 {
   double pred;
   double result;
@@ -81,9 +95,8 @@
     {
       return GSL_NAN;
     }
-  
   pred = pspp_linreg_predict (predictors, vals, c, n_vals);
-  
+
   result = gsl_isnan (pred) ? GSL_NAN : (obs->f - pred);
   return result;
 }