Categoricals: remove var member which is no longer required
authorJohn Darrington <john@darrington.wattle.id.au>
Thu, 7 Jul 2011 09:39:21 +0000 (11:39 +0200)
committerJohn Darrington <john@darrington.wattle.id.au>
Tue, 26 Jul 2011 10:36:37 +0000 (12:36 +0200)
src/math/categoricals.c
src/math/categoricals.h
src/math/covariance.c
src/math/covariance.h

index 16bb46f6a1cbc72601c8e70b5e44c2f04508898f..58015c9088c891653329765a9cfc215f67b6b9e8 100644 (file)
@@ -48,7 +48,6 @@ struct var_params
   /* A map indexed by a union values */
   struct hmap map;
 
-  const struct variable *var;
   const struct interaction *iact;
 
   int base_subscript_short;
@@ -82,7 +81,6 @@ struct categoricals
   /* The weight variable */
   const struct variable *wv;
 
-
   /* An array of var_params */
   struct var_params *vp;
 
@@ -244,7 +242,6 @@ categoricals_create (const struct interaction **inter, size_t n_inter,
   for (i = 0 ; i < cat->n_vp; ++i)
     {
       hmap_init (&cat->vp[i].map);
-      cat->vp[i].var = inter[i]->vars[0];
       cat->vp[i].iact = inter[i];
     }
 
@@ -266,23 +263,17 @@ categoricals_update (struct categoricals *cat, const struct ccase *c)
   for (i = 0 ; i < cat->n_vp; ++i)
     {
       const struct interaction *iact = cat->vp[i].iact;
-      const struct variable *var = cat->vp[i].var;
-
-      const union value *val = case_data (c, var);
       size_t hash;
       struct value_node *node ;
 
-#if XXX
-      if ( var_is_value_missing (var, val, cat->exclude))
+      if ( interaction_case_is_missing (iact, c, cat->exclude))
        continue;
-#endif
 
       hash = interaction_case_hash (iact, c);
       node = lookup_case (&cat->vp[i].map, iact, c);
 
       if ( NULL == node)
        {
-         int width  = var_get_width (var);
          node = pool_malloc (cat->pool, sizeof *node);
 
          node->ccase = case_ref (c);
@@ -304,7 +295,7 @@ categoricals_update (struct categoricals *cat, const struct ccase *c)
       cat->vp[i].cc += weight;
 
       if (cat->update)
-       cat->update (node->user_data, cat->exclude, cat->wv, var, c, cat->aux1, cat->aux2);
+       cat->update (node->user_data, cat->exclude, cat->wv, NULL, c, cat->aux1, cat->aux2);
     }
 }
 
@@ -409,11 +400,11 @@ categoricals_get_variable_by_subscript (const struct categoricals *cat, int subs
 {
   int index = reverse_variable_lookup_short (cat, subscript);
 
-  return cat->vp[index].var;
+  return cat->vp[index].iact->vars[0];
 }
 
 /* Return the interaction corresponding to SUBSCRIPT */
-static const struct interaction *
+const struct interaction *
 categoricals_get_interaction_by_subscript (const struct categoricals *cat, int subscript)
 {
   int index = reverse_variable_lookup_short (cat, subscript);
index 2cd55b5ec3afa1a6d8630d754bf682518c817c56..1a698dea72c247f48de1f183734bc82db27a33b8 100644 (file)
@@ -79,6 +79,7 @@ void categoricals_done (const struct categoricals *cat);
 const struct variable * categoricals_get_variable_by_subscript (const struct categoricals *cat, int subscript);
 
 double categoricals_get_weight_by_subscript (const struct categoricals *cat, int subscript);
+const struct interaction *categoricals_get_interaction_by_subscript (const struct categoricals *cat, int subscript);
 
 double categoricals_get_sum_by_subscript (const struct categoricals *cat, int subscript);
 
index adad3439e056e0baad8c35b515972c89747b8ac0..aed42436ba40d95c240ef095960cae2862efb624 100644 (file)
@@ -756,3 +756,18 @@ covariance_get_var_indices (const struct covariance *cov, const struct variable
       vars[i] = categoricals_get_variable_by_subscript (cov->categoricals, i - cov->n_vars);
     }
 }
+
+
+void
+covariance_get_interaction_indices (const struct covariance *cov, const struct interaction **iacts)
+{
+  int i;
+  for (i = 0; i < cov->n_vars; i++)
+    {
+      iacts[i] = cov->vars[i];
+    }
+  for (i = cov->n_vars; i < cov->dim; i++)
+    {
+      iacts[i] = categoricals_get_interaction_by_subscript (cov->categoricals, i - cov->n_vars);
+    }
+}
index 0231f7989a0b07b39e5cf1e4c93492736ee66374..87afd8b28210469f66fb9eb5edff9bb33af72235 100644 (file)
@@ -47,6 +47,8 @@ void covariance_destroy (struct covariance *cov);
 const gsl_matrix *covariance_moments (const struct covariance *cov, int m);
 
 const struct categoricals * covariance_get_categoricals (const struct covariance *cov);
+struct interaction;
+void covariance_get_interaction_indices (const struct covariance *cov, const struct interaction **iacts);
 
 void covariance_get_var_indices (const struct covariance *cov, const struct variable **vars);
 size_t covariance_dim (const struct covariance * cov);