X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Flanguage%2Fstats%2Fbinomial.c;fp=src%2Flanguage%2Fstats%2Fbinomial.c;h=f4344b76e86010603e10e0af33a12e8f717632af;hb=7fbfc32fc3c636959b0a25b3e76609f86519e84a;hp=7a0ac72292e802587312f43cfe8eac18894c21cc;hpb=da299bd9871b178336a440c6ac53aebc3cea672e;p=pspp-builds.git

diff --git a/src/language/stats/binomial.c b/src/language/stats/binomial.c
index 7a0ac722..f4344b76 100644
--- a/src/language/stats/binomial.c
+++ b/src/language/stats/binomial.c
@@ -1,5 +1,5 @@
 /* PSPP - a program for statistical analysis.
-   Copyright (C) 2006 Free Software Foundation, Inc.
+   Copyright (C) 2006, 2009 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
@@ -99,41 +99,51 @@ do_binomial (const struct dictionary *dict,
   bool warn = true;
 
   const struct one_sample_test *ost = (const struct one_sample_test *) bst;
-  struct ccase c;
+  struct ccase *c;
 
-  while (casereader_read(input, &c))
+  while ((c = casereader_read(input)) != NULL)
     {
       int v;
-      double w = dict_get_case_weight (dict, &c, &warn);
+      double w = dict_get_case_weight (dict, c, &warn);
 
       for (v = 0 ; v < ost->n_vars ; ++v )
 	{
 	  const struct variable *var = ost->vars[v];
-	  const union value *value = case_data (&c, var);
+	  const union value *value = case_data (c, var);
           int width = var_get_width (var);
 
 	  if (var_is_value_missing (var, value, exclude))
-	    break;
+	    continue;
 
-	  if ( NULL == cat1[v].value )
+	  if (bst->cutpoint != SYSMIS)
 	    {
-	      cat1[v].value = value_dup (value, width);
-	      cat1[v].count = w;
+	      if ( compare_values_short (cat1[v].value, value, var) >= 0 )
+		  cat1[v].count  += w;
+	      else
+		  cat2[v].count += w;
 	    }
-	  else if ( 0 == compare_values (cat1[v].value, value, width))
-	    cat1[v].count += w;
-	  else if ( NULL == cat2[v].value )
+	  else
 	    {
-	      cat2[v].value = value_dup (value, width);
-	      cat2[v].count = w;
+	      if ( NULL == cat1[v].value )
+		{
+		  cat1[v].value = value_dup (value, width);
+		  cat1[v].count = w;
+		}
+	      else if ( 0 == compare_values_short (cat1[v].value, value, var))
+		cat1[v].count += w;
+	      else if ( NULL == cat2[v].value )
+		{
+		  cat2[v].value = value_dup (value, width);
+		  cat2[v].count = w;
+		}
+	      else if ( 0 == compare_values_short (cat2[v].value, value, var))
+		cat2[v].count += w;
+	      else if ( bst->category1 == SYSMIS)
+		msg (ME, _("Variable %s is not dichotomous"), var_get_name (var));
 	    }
-	  else if ( 0 == compare_values (cat2[v].value, value, width))
-	    cat2[v].count += w;
-	  else if ( bst->category1 == SYSMIS)
-	    msg (ME, _("Variable %s is not dichotomous"), var_get_name (var));
 	}
 
-      case_destroy (&c);
+      case_unref (c);
     }
   return casereader_destroy (input);
 }
@@ -144,7 +154,9 @@ void
 binomial_execute (const struct dataset *ds,
 		  struct casereader *input,
                   enum mv_class exclude,
-		  const struct npar_test *test)
+		  const struct npar_test *test,
+		  bool exact UNUSED,
+		  double timer UNUSED)
 {
   int v;
   const struct dictionary *dict = dataset_dict (ds);
@@ -154,20 +166,32 @@ binomial_execute (const struct dataset *ds,
   struct freq_mutable *cat1 = xzalloc (sizeof (*cat1) * ost->n_vars);
   struct freq_mutable *cat2 = xzalloc (sizeof (*cat1) * ost->n_vars);
 
-  assert ((bst->category1 == SYSMIS) == (bst->category2 == SYSMIS) );
+  assert ((bst->category1 == SYSMIS) == (bst->category2 == SYSMIS) || bst->cutpoint != SYSMIS);
 
-  if ( bst->category1 != SYSMIS )
+  if ( bst->cutpoint != SYSMIS )
     {
+      int i;
+      union value v;
+      v.f = bst->cutpoint;
+      for (i = 0; i < ost->n_vars; i++)
+	cat1[i].value = value_dup (&v, 0);
+    }
+  else  if ( bst->category1 != SYSMIS )
+    {
+      int i;
       union value v;
       v.f = bst->category1;
-      cat1->value = value_dup (&v, 0);
+      for (i = 0; i < ost->n_vars; i++)
+	cat1[i].value = value_dup (&v, 0);
     }
 
   if ( bst->category2 != SYSMIS )
     {
+      int i;
       union value v;
       v.f = bst->category2;
-      cat2->value = value_dup (&v, 0);
+      for (i = 0; i < ost->n_vars; i++)
+	cat2[i].value = value_dup (&v, 0);
     }
 
   if (do_binomial (dict, input, bst, cat1, cat2, exclude))
@@ -197,8 +221,15 @@ binomial_execute (const struct dataset *ds,
 	  ds_init_empty (&catstr1);
 	  ds_init_empty (&catstr2);
 
-	  var_append_value_name (var, cat1[v].value, &catstr1);
-	  var_append_value_name (var, cat2[v].value, &catstr2);
+	  if ( bst->cutpoint != SYSMIS)
+	    {
+	      ds_put_format (&catstr1, "<= %g", bst->cutpoint);
+	    }
+	  else
+	    {
+	      var_append_value_name (var, cat1[v].value, &catstr1);
+	      var_append_value_name (var, cat2[v].value, &catstr2);
+	    }
 
           tab_hline (table, TAL_1, 0, tab_nc (table) -1, 1 + v * 3);
 
@@ -229,7 +260,7 @@ binomial_execute (const struct dataset *ds,
                      cat2[v].count / n_total, NULL);
 
           tab_double (table, 4, 3 + v * 3, TAB_NONE,
-                     (cat1[v].count + cat2[v].count) / n_total, wfmt);
+                     (cat1[v].count + cat2[v].count) / n_total, NULL);
 
           /* Significance */
           sig = calculate_binomial (cat1[v].count, cat2[v].count, bst->p);