[GRASS-SVN] r53277 - grass/trunk/vector/v.univar

svn_grass at osgeo.org svn_grass at osgeo.org
Thu Sep 27 11:01:51 PDT 2012


Author: mmetz
Date: 2012-09-27 11:01:50 -0700 (Thu, 27 Sep 2012)
New Revision: 53277

Modified:
   grass/trunk/vector/v.univar/main.c
Log:
v.univar: forward port new -d flag, some fixes for -d flag (to be backported)

Modified: grass/trunk/vector/v.univar/main.c
===================================================================
--- grass/trunk/vector/v.univar/main.c	2012-09-27 15:44:17 UTC (rev 53276)
+++ grass/trunk/vector/v.univar/main.c	2012-09-27 18:01:50 UTC (rev 53277)
@@ -16,6 +16,28 @@
  *               comes with GRASS for details.
  *
  **************************************************************/
+
+/* TODO
+ * - add flag to weigh by line/boundary length and area size
+ *   Roger Bivand on GRASS devel ml on July 2 2004
+ *   http://lists.osgeo.org/pipermail/grass-dev/2004-July/014976.html
+ *   "[...] calculating weighted means, weighting by line length
+ *   or area surface size [does not make sense]. I think it would be
+ *   better to treat each line or area as a discrete, unweighted, unit
+ *   unless some reason to the contrary is given, just like points/sites.
+ *   It is probably more important to handle missing data gracefully
+ *   than weight the means or other statistics, I think. There may be
+ *   reasons to weigh sometimes, but most often I see ratios or rates of
+ *   two variables, rather than of a single variable and length or area."
+ * 
+ * - use geodesic distances for the geometry option (distance to closest
+ *   other feature)
+ * 
+ * - use sample instead of population mean/stddev
+ * */
+
+
+
 #include <stdlib.h>
 #include <string.h>
 #include <math.h>
@@ -24,41 +46,46 @@
 #include <grass/dbmi.h>
 #include <grass/glocale.h>
 
+static void select_from_geometry(void);
+static void select_from_database(void);
+static void summary(void);
+
+struct Option *field_opt, *where_opt, *col_opt;
+struct Flag *shell_flag, *extended, *geometry;
+struct Map_info Map;
+struct line_cats *Cats;
+struct field_info *Fi;
+dbDriver *Driver;
+dbCatValArray Cvarr;
+int otype, ofield;
+int compatible = 1;		/* types are compatible: point+centroid or line+boundary or area */
+int nmissing = 0;		/* number of missing atttributes */
+int nnull = 0;		/* number of null values */
+int nzero = 0;		/* number of zero distances */
+int first = 1;
+
+/* Statistics */
+int count = 0;		/* number of features with non-null attribute */
+double sum = 0.0;
+double sumsq = 0.0;
+double sumcb = 0.0;
+double sumqt = 0.0;
+double sum_abs = 0.0;
+double min = 0.0 / 0.0;	/* init as nan */
+double max = 0.0 / 0.0;
+double mean, mean_abs, pop_variance, sample_variance, pop_stdev,
+    sample_stdev, pop_coeff_variation, kurtosis, skewness;
+double total_size = 0.0;	/* total size: length/area */
+
+/* Extended statistics */
+int perc;
+
 int main(int argc, char *argv[])
 {
     struct GModule *module;
-    struct Option *map_opt, *type_opt, *field_opt, *col_opt, *where_opt,
+    struct Option *map_opt, *type_opt,
 	*percentile;
-    struct Flag *shell_flag, *extended;
-    struct Map_info Map;
-    struct field_info *Fi;
-    dbDriver *Driver;
-    dbCatValArray Cvarr;
-    struct line_pnts *Points;
-    struct line_cats *Cats;
-    int otype, ofield;
-    int compatible = 1;		/* types are compatible: point+centroid or line+boundary or area */
-    int nrec, ctype, nlines, line, nareas, area;
-    int nmissing = 0;		/* number of missing atttributes */
-    int nnull = 0;		/* number of null values */
-    int first = 1;
 
-    /* Statistics */
-    int count = 0;		/* number of features with non-null attribute */
-    double sum = 0.0;
-    double sumsq = 0.0;
-    double sumcb = 0.0;
-    double sumqt = 0.0;
-    double sum_abs = 0.0;
-    double min = 0.0 / 0.0;	/* init as nan */
-    double max = 0.0 / 0.0;
-    double mean, mean_abs, pop_variance, sample_variance, pop_stdev,
-	sample_stdev, pop_coeff_variation, kurtosis, skewness;
-    double total_size = 0.0;	/* total size: length/area */
-
-    /* Extended statistics */
-    int perc;
-
     module = G_define_module();
     G_add_keyword(_("vector"));
     G_add_keyword(_("statistics"));
@@ -76,7 +103,7 @@
     type_opt->answer = "point,line,area";
 
     col_opt = G_define_standard_option(G_OPT_DB_COLUMN);
-    col_opt->required = YES;
+    col_opt->required = NO;
 
     where_opt = G_define_standard_option(G_OPT_DB_WHERE);
 
@@ -97,15 +124,23 @@
     extended->key = 'e';
     extended->description = _("Calculate extended statistics");
 
+    geometry = G_define_flag();
+    geometry->key = 'd';
+    geometry->description = _("Calculate geometry distances instead of table data.");
+
     G_gisinit(argv[0]);
 
     if (G_parser(argc, argv))
 	exit(EXIT_FAILURE);
 
+    /* Only require col_opt answer if -g flag is not set */
+    if (!col_opt->answer && !geometry->answer) {
+	G_fatal_error(_("Required parameter <%s> not set:\n\t(%s)"), col_opt->key, col_opt->description);
+    }
+
     otype = Vect_option_to_types(type_opt);
     perc = atoi(percentile->answer);
 
-    Points = Vect_new_line_struct();
     Cats = Vect_new_cats_struct();
 
     /* open input vector */
@@ -118,18 +153,138 @@
 	compatible = 0;
     if ((otype & GV_LINES) && (otype & GV_AREA))
 	compatible = 0;
+    if (!compatible && geometry->answer)
+	compatible = 1; /* distances is compatible with all kinds of geometries */
 
     if (!compatible) {
 	G_warning(_("Incompatible vector type(s) specified, only number of features, minimum, maximum and range "
 		   "can be calculated"));
     }
 
-    if (extended->answer && !(otype & GV_POINTS)) {
+    if (extended->answer && (!(otype & GV_POINTS) || geometry->answer)) {
 	G_warning(_("Extended statistics is currently supported only for points/centroids"));
     }
 
-    /* Read attributes */
-    db_CatValArray_init(&Cvarr);
+    if (geometry->answer)
+	select_from_geometry();
+    else
+	select_from_database();
+    summary();
+
+    Vect_close(&Map);
+
+    exit(EXIT_SUCCESS);
+}
+
+static void select_from_geometry(void)
+{
+    int i, j, k, ncats, *cats;
+    int type;
+    struct line_pnts *iPoints, *jPoints;
+    int nlines;
+
+    iPoints = Vect_new_line_struct();
+    jPoints = Vect_new_line_struct();
+
+    if (where_opt->answer != NULL) {
+	if (ofield < 1) {
+	    G_fatal_error(_("'layer' must be > 0 for 'where'."));
+	}
+	Fi = Vect_get_field(&Map, ofield);
+	if (!Fi) {
+	    G_fatal_error(_("Database connection not defined for layer %d"),
+			  ofield);
+	}
+
+	Driver = db_start_driver_open_database(Fi->driver, Fi->database);
+	if (Driver == NULL)
+	    G_fatal_error("Unable to open database <%s> by driver <%s>",
+			  Fi->database, Fi->driver);
+	ncats = db_select_int(Driver, Fi->table, Fi->key, where_opt->answer,
+			      &cats);
+	if (ncats == -1)
+		G_fatal_error(_("Unable select categories from table <%s>"), Fi->table);
+
+	db_close_database_shutdown_driver(Driver);
+
+    }
+    else
+	ncats = 0;
+
+    count = 0;
+
+    nlines = Vect_get_num_lines(&Map);
+    /* Start calculating the statistics based on distance to all other primitives.
+       Use the centroid of areas and the first point of lines */
+    for (i = 1; i <= nlines; i++) {
+
+	type = Vect_read_line(&Map, iPoints, Cats, i);
+
+	if (!(type & otype))
+	    continue;
+
+	if (where_opt->answer) {
+	    int ok = FALSE;
+
+	    for (j = 0; j < Cats->n_cats; j++) {
+		if (Vect_cat_in_array(Cats->cat[j], cats, ncats)) {
+		    ok = TRUE;
+		    break;
+		}
+	    }
+	    if (!ok)
+	        continue;
+	}
+	for (j = i + 1; j < nlines; j++) {
+	    /* get distance to this object */
+	    double val = 0.0;
+
+	    type = Vect_read_line(&Map, jPoints, Cats, j);
+
+	    if (!(type & otype))
+		continue;
+
+	    /* now calculate the min distance between each point in line i with line j */
+	    for (k = 0; k < iPoints->n_points; k++) {
+		double dmin = 0.0;
+
+		Vect_line_distance(jPoints, iPoints->x[k], iPoints->y[k], iPoints->z[k], 1,
+				       NULL, NULL, NULL, &dmin, NULL, NULL);
+		if((k == 0) || (dmin < val)) {
+		    val = dmin;
+		}
+	    }
+	    if (val == 0) {
+		nzero++;
+		continue;
+	    }
+	    if (first) {
+		max = val;
+		min = val;
+		first = 0;
+	    }
+	    else {
+		if (val > max)
+		    max = val;
+		if (val < min)
+		    min = val;
+	    }
+	    sum += val;
+	    sumsq += val * val;
+	    sumcb += val * val * val;
+	    sumqt += val * val * val * val;
+	    sum_abs += fabs(val);
+	    count++;
+	    G_debug(3, "i=%d j=%d sum = %f val=%f", i, j, sum, val);
+	}
+    }
+}
+
+static void select_from_database(void)
+{
+    int nrec, ctype, nlines, line, nareas, area;
+    struct line_pnts *Points;
+
     Fi = Vect_get_field(&Map, ofield);
     if (Fi == NULL) {
 	G_fatal_error(_(" Database connection not defined for layer <%s>"), field_opt->answer);
@@ -141,7 +296,7 @@
 		      Fi->database, Fi->driver);
 
     /* Note do not check if the column exists in the table because it may be an expression */
-
+    db_CatValArray_init(&Cvarr);
     nrec =
 	db_select_CatValArray(Driver, Fi->table, Fi->key, col_opt->answer,
 			      where_opt->answer, &Cvarr);
@@ -156,6 +311,8 @@
 
     db_close_database_shutdown_driver(Driver);
 
+    Points = Vect_new_line_struct();
+
     /* Lines */
     nlines = Vect_get_num_lines(&Map);
 
@@ -307,9 +464,12 @@
     }
 
     G_debug(2, "sum = %f total_size = %f", sum, total_size);
+}
 
+static void summary(void)
+{
     if (compatible) {
-	if ((otype & GV_LINES) || (otype & GV_AREA)) {
+	if (((otype & GV_LINES) || (otype & GV_AREA)) && !geometry->answer) {
 	    mean = sum / total_size;
 	    mean_abs = sum_abs / total_size;
 	    /* Roger Bivand says it is wrong see GRASS devel list 7/2004 */
@@ -345,8 +505,13 @@
 
     if (shell_flag->answer) {
 	fprintf(stdout, "n=%d\n", count);
-	fprintf(stdout, "nmissing=%d\n", nmissing);
-	fprintf(stdout, "nnull=%d\n", nnull);
+	if (geometry->answer) {
+	    fprintf(stdout, "nzero=%d\n", nzero);
+	}
+	else {
+	    fprintf(stdout, "nmissing=%d\n", nmissing);
+	    fprintf(stdout, "nnull=%d\n", nnull);
+	}
 	if (count > 0) {
 	    fprintf(stdout, "min=%g\n", min);
 	    fprintf(stdout, "max=%g\n", max);
@@ -368,10 +533,16 @@
 	}
     }
     else {
-	fprintf(stdout, "number of features with non NULL attribute: %d\n",
-		count);
-	fprintf(stdout, "number of missing attributes: %d\n", nmissing);
-	fprintf(stdout, "number of NULL attributes: %d\n", nnull);
+	if(geometry->answer) {
+	    fprintf(stdout, "number of non zero distances: %d\n", count);
+	    fprintf(stdout, "number of zero distances: %d\n", nzero);
+	}
+	else {
+	    fprintf(stdout, "number of features with non NULL attribute: %d\n",
+		    count);
+	    fprintf(stdout, "number of missing attributes: %d\n", nmissing);
+	    fprintf(stdout, "number of NULL attributes: %d\n", nnull);
+	}
 	if (count > 0) {
 	    fprintf(stdout, "minimum: %g\n", min);
 	    fprintf(stdout, "maximum: %g\n", max);
@@ -396,7 +567,8 @@
     }
 
     /* TODO: mode, skewness, kurtosis */
-    if (extended->answer && compatible && (otype & GV_POINTS) && count > 0) {
+    /* Not possible to calculate for point distance, since we don't collect the population */
+    if (extended->answer && compatible && (otype & GV_POINTS) && !geometry->answer && count > 0) {
 	double quartile_25 = 0.0, quartile_75 = 0.0, quartile_perc = 0.0;
 	double median = 0.0;
 	int qpos_25, qpos_75, qpos_perc;
@@ -456,8 +628,4 @@
 		fprintf(stdout, "%dth percentile: %g\n", perc, quartile_perc);
 	}
     }
-
-    Vect_close(&Map);
-
-    exit(EXIT_SUCCESS);
 }



More information about the grass-commit mailing list