Rename src/output/charts/box-whisker.[ch] to src/output/charts/boxplot.[ch]. fc11-i386-build54 fc11-x64-build51 lenny-x64-build75 sid-i386-build121
authorBen Pfaff <blp@gnu.org>
Mon, 14 Dec 2009 05:39:18 +0000 (21:39 -0800)
committerBen Pfaff <blp@gnu.org>
Tue, 15 Dec 2009 05:18:17 +0000 (21:18 -0800)
This makes the structure names more obvious and prevents confusion
with src/math/box-whisker.[ch].

src/language/stats/examine.q
src/output/automake.mk
src/output/charts/box-whisker.c [deleted file]
src/output/charts/box-whisker.h [deleted file]
src/output/charts/boxplot.c [new file with mode: 0644]
src/output/charts/boxplot.h [new file with mode: 0644]

index 1d399b5a1888908a4b08c8cfe2874da0af039bed..4ca3af966505080338b2ff85679885ed1bd7a963 100644 (file)
@@ -48,7 +48,7 @@
 #include <libpspp/misc.h>
 #include <libpspp/str.h>
 #include <math/moments.h>
-#include <output/charts/box-whisker.h>
+#include <output/charts/boxplot.h>
 #include <output/charts/np-plot.h>
 #include <output/manager.h>
 #include <output/table.h>
index ad8e4529c8cf0695f2790abac5a5322ff4c0ade5..7e233750b8f3c5652206e3fb846250ba4947610c 100644 (file)
@@ -8,8 +8,8 @@ src_output_liboutput_la_SOURCES = \
        src/output/ascii.c \
        src/output/chart.c \
        src/output/chart.h \
-       src/output/charts/box-whisker.c \
-       src/output/charts/box-whisker.h \
+       src/output/charts/boxplot.c \
+       src/output/charts/boxplot.h \
        src/output/charts/cartesian.c \
        src/output/charts/cartesian.h \
        src/output/charts/np-plot.c \
diff --git a/src/output/charts/box-whisker.c b/src/output/charts/box-whisker.c
deleted file mode 100644 (file)
index 5fd9d29..0000000
+++ /dev/null
@@ -1,263 +0,0 @@
-/* PSPP - a program for statistical analysis.
-   Copyright (C) 2004, 2008, 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
-   the Free Software Foundation, either version 3 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, see <http://www.gnu.org/licenses/>. */
-
-
-#include <config.h>
-
-#include <output/charts/box-whisker.h>
-
-#include <math.h>
-#include <assert.h>
-#include <cairo/cairo.h>
-
-#include <libpspp/cast.h>
-#include <libpspp/misc.h>
-#include <math/chart-geometry.h>
-#include <math/box-whisker.h>
-#include <output/chart.h>
-#include <output/chart-provider.h>
-#include <output/charts/plot-chart.h>
-
-/* Draw a box-and-whiskers plot
-*/
-
-struct box
-  {
-    struct box_whisker *bw;
-    char *label;
-  };
-
-struct boxplot
-  {
-    struct chart chart;
-    double y_min;
-    double y_max;
-    char *title;
-    struct box *boxes;
-    size_t n_boxes, boxes_allocated;
-  };
-
-static const struct chart_class boxplot_chart_class;
-
-struct boxplot *
-boxplot_create (double y_min, double y_max, const char *title)
-{
-  struct boxplot *boxplot = xmalloc (sizeof *boxplot);
-  chart_init (&boxplot->chart, &boxplot_chart_class);
-  boxplot->y_min = y_min;
-  boxplot->y_max = y_max;
-  boxplot->title = xstrdup (title);
-  boxplot->boxes = NULL;
-  boxplot->n_boxes = boxplot->boxes_allocated = 0;
-  return boxplot;
-}
-
-void
-boxplot_add_box (struct boxplot *boxplot,
-                 struct box_whisker *bw, const char *label)
-{
-  struct box *box;
-  if (boxplot->n_boxes >= boxplot->boxes_allocated)
-    boxplot->boxes = x2nrealloc (boxplot->boxes, &boxplot->boxes_allocated,
-                                 sizeof *boxplot->boxes);
-  box = &boxplot->boxes[boxplot->n_boxes++];
-  box->bw = bw;
-  box->label = xstrdup (label);
-}
-
-struct chart *
-boxplot_get_chart (struct boxplot *boxplot)
-{
-  return &boxplot->chart;
-}
-
-/* Draw an OUTLIER on the plot CH
- * at CENTRELINE
- */
-static void
-draw_case (cairo_t *cr, const struct chart_geometry *geom, double centreline,
-          const struct outlier *outlier)
-{
-  double y = geom->data_bottom + (outlier->value - geom->y_min) * geom->ordinate_scale;
-  chart_draw_marker (cr, centreline, y,
-                     outlier->extreme ? MARKER_ASTERISK : MARKER_CIRCLE,
-                     20);
-
-  cairo_move_to (cr, centreline + 10, y);
-  chart_label (cr, 'l', 'c', geom->font_size, ds_cstr (&outlier->label));
-}
-
-static void
-boxplot_draw_box (cairo_t *cr, const struct chart_geometry *geom,
-                  double box_centre,
-                  double box_width,
-                  const struct box_whisker *bw,
-                  const char *name)
-{
-  double whisker[2];
-  double hinge[3];
-  struct ll *ll;
-
-  const struct ll_list *outliers;
-
-  const double box_left = box_centre - box_width / 2.0;
-
-  const double box_right = box_centre + box_width / 2.0;
-
-  double box_bottom ;
-  double box_top ;
-  double bottom_whisker ;
-  double top_whisker ;
-
-  box_whisker_whiskers (bw, whisker);
-  box_whisker_hinges (bw, hinge);
-
-  box_bottom = geom->data_bottom + (hinge[0] - geom->y_min ) * geom->ordinate_scale;
-
-  box_top = geom->data_bottom + (hinge[2] - geom->y_min ) * geom->ordinate_scale;
-
-  bottom_whisker = geom->data_bottom + (whisker[0] - geom->y_min) *
-    geom->ordinate_scale;
-
-  top_whisker = geom->data_bottom + (whisker[1] - geom->y_min) * geom->ordinate_scale;
-
-  /* Draw the box */
-  cairo_rectangle (cr,
-                   box_left,
-                   box_bottom,
-                   box_right - box_left,
-                   box_top - box_bottom);
-  cairo_save (cr);
-  cairo_set_source_rgb (cr,
-                        geom->fill_colour.red / 255.0,
-                        geom->fill_colour.green / 255.0,
-                        geom->fill_colour.blue / 255.0);
-  cairo_fill (cr);
-  cairo_restore (cr);
-  cairo_stroke (cr);
-
-  /* Draw the median */
-  cairo_save (cr);
-  cairo_set_line_width (cr, cairo_get_line_width (cr) * 5);
-  cairo_move_to (cr,
-                 box_left,
-                 geom->data_bottom + (hinge[1] - geom->y_min) * geom->ordinate_scale);
-  cairo_line_to (cr,
-                 box_right,
-                 geom->data_bottom + (hinge[1] - geom->y_min) * geom->ordinate_scale);
-  cairo_stroke (cr);
-  cairo_restore (cr);
-
-  /* Draw the bottom whisker */
-  cairo_move_to (cr, box_left, bottom_whisker);
-  cairo_line_to (cr, box_right, bottom_whisker);
-  cairo_stroke (cr);
-
-  /* Draw top whisker */
-  cairo_move_to (cr, box_left, top_whisker);
-  cairo_line_to (cr, box_right, top_whisker);
-  cairo_stroke (cr);
-
-  /* Draw centre line.
-     (bottom half) */
-  cairo_move_to (cr, box_centre, bottom_whisker);
-  cairo_line_to (cr, box_centre, box_bottom);
-  cairo_stroke (cr);
-
-  /* (top half) */
-  cairo_move_to (cr, box_centre, top_whisker);
-  cairo_line_to (cr, box_centre, box_top);
-  cairo_stroke (cr);
-
-  outliers = box_whisker_outliers (bw);
-  for (ll = ll_head (outliers);
-       ll != ll_null (outliers); ll = ll_next (ll))
-    {
-      const struct outlier *outlier = ll_data (ll, struct outlier, ll);
-      draw_case (cr, geom, box_centre, outlier);
-    }
-
-  /* Draw  tick  mark on x axis */
-  draw_tick(cr, geom, TICK_ABSCISSA, box_centre - geom->data_left, "%s", name);
-}
-
-static void
-boxplot_draw_yscale (cairo_t *cr, struct chart_geometry *geom,
-                     double y_max, double y_min)
-{
-  double y_tick;
-  double d;
-
-  geom->y_max = y_max;
-  geom->y_min = y_min;
-
-  y_tick = chart_rounded_tick (fabs (geom->y_max - geom->y_min) / 5.0);
-
-  geom->y_min = (ceil (geom->y_min / y_tick) - 1.0) * y_tick;
-
-  geom->y_max = (floor (geom->y_max / y_tick) + 1.0) * y_tick;
-
-  geom->ordinate_scale = (fabs (geom->data_top - geom->data_bottom)
-                          / fabs (geom->y_max - geom->y_min));
-
-  for (d = geom->y_min; d <= geom->y_max; d += y_tick)
-    draw_tick (cr, geom, TICK_ORDINATE,
-               (d - geom->y_min) * geom->ordinate_scale, "%g", d);
-}
-
-static void
-boxplot_chart_draw (const struct chart *chart, cairo_t *cr,
-                    struct chart_geometry *geom)
-{
-  const struct boxplot *boxplot = UP_CAST (chart, struct boxplot, chart);
-  double box_width;
-  size_t i;
-
-  boxplot_draw_yscale (cr, geom, boxplot->y_max, boxplot->y_min);
-  chart_write_title (cr, geom, "%s", boxplot->title);
-
-  box_width = (geom->data_right - geom->data_left) / boxplot->n_boxes / 2.0;
-  for (i = 0; i < boxplot->n_boxes; i++)
-    {
-      const struct box *box = &boxplot->boxes[i];
-      const double box_centre = (i * 2 + 1) * box_width + geom->data_left;
-      boxplot_draw_box (cr, geom, box_centre, box_width, box->bw, box->label);
-    }
-}
-
-static void
-boxplot_chart_destroy (struct chart *chart)
-{
-  struct boxplot *boxplot = UP_CAST (chart, struct boxplot, chart);
-  size_t i;
-
-  free (boxplot->title);
-  for (i = 0; i < boxplot->n_boxes; i++)
-    {
-      struct box *box = &boxplot->boxes[i];
-      struct statistic *statistic = &box->bw->parent.parent;
-      statistic->destroy (statistic);
-      free (box->label);
-    }
-  free (boxplot->boxes);
-  free (boxplot);
-}
-
-static const struct chart_class boxplot_chart_class =
-  {
-    boxplot_chart_draw,
-    boxplot_chart_destroy
-  };
diff --git a/src/output/charts/box-whisker.h b/src/output/charts/box-whisker.h
deleted file mode 100644 (file)
index 67c1e12..0000000
+++ /dev/null
@@ -1,27 +0,0 @@
-/* PSPP - a program for statistical analysis.
-   Copyright (C) 2004, 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
-   the Free Software Foundation, either version 3 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, see <http://www.gnu.org/licenses/>. */
-
-#ifndef BOX_WHISKER_H
-#define BOX_WHISKER_H
-
-struct box_whisker;
-
-struct boxplot *boxplot_create (double y_min, double y_max, const char *title);
-void boxplot_add_box (struct boxplot *,
-                      struct box_whisker *, const char *label);
-struct chart *boxplot_get_chart (struct boxplot *);
-
-#endif
diff --git a/src/output/charts/boxplot.c b/src/output/charts/boxplot.c
new file mode 100644 (file)
index 0000000..015385e
--- /dev/null
@@ -0,0 +1,263 @@
+/* PSPP - a program for statistical analysis.
+   Copyright (C) 2004, 2008, 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
+   the Free Software Foundation, either version 3 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, see <http://www.gnu.org/licenses/>. */
+
+
+#include <config.h>
+
+#include <output/charts/boxplot.h>
+
+#include <math.h>
+#include <assert.h>
+#include <cairo/cairo.h>
+
+#include <libpspp/cast.h>
+#include <libpspp/misc.h>
+#include <math/chart-geometry.h>
+#include <math/box-whisker.h>
+#include <output/chart.h>
+#include <output/chart-provider.h>
+#include <output/charts/plot-chart.h>
+
+/* Draw a box-and-whiskers plot
+*/
+
+struct box
+  {
+    struct box_whisker *bw;
+    char *label;
+  };
+
+struct boxplot
+  {
+    struct chart chart;
+    double y_min;
+    double y_max;
+    char *title;
+    struct box *boxes;
+    size_t n_boxes, boxes_allocated;
+  };
+
+static const struct chart_class boxplot_chart_class;
+
+struct boxplot *
+boxplot_create (double y_min, double y_max, const char *title)
+{
+  struct boxplot *boxplot = xmalloc (sizeof *boxplot);
+  chart_init (&boxplot->chart, &boxplot_chart_class);
+  boxplot->y_min = y_min;
+  boxplot->y_max = y_max;
+  boxplot->title = xstrdup (title);
+  boxplot->boxes = NULL;
+  boxplot->n_boxes = boxplot->boxes_allocated = 0;
+  return boxplot;
+}
+
+void
+boxplot_add_box (struct boxplot *boxplot,
+                 struct box_whisker *bw, const char *label)
+{
+  struct box *box;
+  if (boxplot->n_boxes >= boxplot->boxes_allocated)
+    boxplot->boxes = x2nrealloc (boxplot->boxes, &boxplot->boxes_allocated,
+                                 sizeof *boxplot->boxes);
+  box = &boxplot->boxes[boxplot->n_boxes++];
+  box->bw = bw;
+  box->label = xstrdup (label);
+}
+
+struct chart *
+boxplot_get_chart (struct boxplot *boxplot)
+{
+  return &boxplot->chart;
+}
+
+/* Draw an OUTLIER on the plot CH
+ * at CENTRELINE
+ */
+static void
+draw_case (cairo_t *cr, const struct chart_geometry *geom, double centreline,
+          const struct outlier *outlier)
+{
+  double y = geom->data_bottom + (outlier->value - geom->y_min) * geom->ordinate_scale;
+  chart_draw_marker (cr, centreline, y,
+                     outlier->extreme ? MARKER_ASTERISK : MARKER_CIRCLE,
+                     20);
+
+  cairo_move_to (cr, centreline + 10, y);
+  chart_label (cr, 'l', 'c', geom->font_size, ds_cstr (&outlier->label));
+}
+
+static void
+boxplot_draw_box (cairo_t *cr, const struct chart_geometry *geom,
+                  double box_centre,
+                  double box_width,
+                  const struct box_whisker *bw,
+                  const char *name)
+{
+  double whisker[2];
+  double hinge[3];
+  struct ll *ll;
+
+  const struct ll_list *outliers;
+
+  const double box_left = box_centre - box_width / 2.0;
+
+  const double box_right = box_centre + box_width / 2.0;
+
+  double box_bottom ;
+  double box_top ;
+  double bottom_whisker ;
+  double top_whisker ;
+
+  box_whisker_whiskers (bw, whisker);
+  box_whisker_hinges (bw, hinge);
+
+  box_bottom = geom->data_bottom + (hinge[0] - geom->y_min ) * geom->ordinate_scale;
+
+  box_top = geom->data_bottom + (hinge[2] - geom->y_min ) * geom->ordinate_scale;
+
+  bottom_whisker = geom->data_bottom + (whisker[0] - geom->y_min) *
+    geom->ordinate_scale;
+
+  top_whisker = geom->data_bottom + (whisker[1] - geom->y_min) * geom->ordinate_scale;
+
+  /* Draw the box */
+  cairo_rectangle (cr,
+                   box_left,
+                   box_bottom,
+                   box_right - box_left,
+                   box_top - box_bottom);
+  cairo_save (cr);
+  cairo_set_source_rgb (cr,
+                        geom->fill_colour.red / 255.0,
+                        geom->fill_colour.green / 255.0,
+                        geom->fill_colour.blue / 255.0);
+  cairo_fill (cr);
+  cairo_restore (cr);
+  cairo_stroke (cr);
+
+  /* Draw the median */
+  cairo_save (cr);
+  cairo_set_line_width (cr, cairo_get_line_width (cr) * 5);
+  cairo_move_to (cr,
+                 box_left,
+                 geom->data_bottom + (hinge[1] - geom->y_min) * geom->ordinate_scale);
+  cairo_line_to (cr,
+                 box_right,
+                 geom->data_bottom + (hinge[1] - geom->y_min) * geom->ordinate_scale);
+  cairo_stroke (cr);
+  cairo_restore (cr);
+
+  /* Draw the bottom whisker */
+  cairo_move_to (cr, box_left, bottom_whisker);
+  cairo_line_to (cr, box_right, bottom_whisker);
+  cairo_stroke (cr);
+
+  /* Draw top whisker */
+  cairo_move_to (cr, box_left, top_whisker);
+  cairo_line_to (cr, box_right, top_whisker);
+  cairo_stroke (cr);
+
+  /* Draw centre line.
+     (bottom half) */
+  cairo_move_to (cr, box_centre, bottom_whisker);
+  cairo_line_to (cr, box_centre, box_bottom);
+  cairo_stroke (cr);
+
+  /* (top half) */
+  cairo_move_to (cr, box_centre, top_whisker);
+  cairo_line_to (cr, box_centre, box_top);
+  cairo_stroke (cr);
+
+  outliers = box_whisker_outliers (bw);
+  for (ll = ll_head (outliers);
+       ll != ll_null (outliers); ll = ll_next (ll))
+    {
+      const struct outlier *outlier = ll_data (ll, struct outlier, ll);
+      draw_case (cr, geom, box_centre, outlier);
+    }
+
+  /* Draw  tick  mark on x axis */
+  draw_tick(cr, geom, TICK_ABSCISSA, box_centre - geom->data_left, "%s", name);
+}
+
+static void
+boxplot_draw_yscale (cairo_t *cr, struct chart_geometry *geom,
+                     double y_max, double y_min)
+{
+  double y_tick;
+  double d;
+
+  geom->y_max = y_max;
+  geom->y_min = y_min;
+
+  y_tick = chart_rounded_tick (fabs (geom->y_max - geom->y_min) / 5.0);
+
+  geom->y_min = (ceil (geom->y_min / y_tick) - 1.0) * y_tick;
+
+  geom->y_max = (floor (geom->y_max / y_tick) + 1.0) * y_tick;
+
+  geom->ordinate_scale = (fabs (geom->data_top - geom->data_bottom)
+                          / fabs (geom->y_max - geom->y_min));
+
+  for (d = geom->y_min; d <= geom->y_max; d += y_tick)
+    draw_tick (cr, geom, TICK_ORDINATE,
+               (d - geom->y_min) * geom->ordinate_scale, "%g", d);
+}
+
+static void
+boxplot_chart_draw (const struct chart *chart, cairo_t *cr,
+                    struct chart_geometry *geom)
+{
+  const struct boxplot *boxplot = UP_CAST (chart, struct boxplot, chart);
+  double box_width;
+  size_t i;
+
+  boxplot_draw_yscale (cr, geom, boxplot->y_max, boxplot->y_min);
+  chart_write_title (cr, geom, "%s", boxplot->title);
+
+  box_width = (geom->data_right - geom->data_left) / boxplot->n_boxes / 2.0;
+  for (i = 0; i < boxplot->n_boxes; i++)
+    {
+      const struct box *box = &boxplot->boxes[i];
+      const double box_centre = (i * 2 + 1) * box_width + geom->data_left;
+      boxplot_draw_box (cr, geom, box_centre, box_width, box->bw, box->label);
+    }
+}
+
+static void
+boxplot_chart_destroy (struct chart *chart)
+{
+  struct boxplot *boxplot = UP_CAST (chart, struct boxplot, chart);
+  size_t i;
+
+  free (boxplot->title);
+  for (i = 0; i < boxplot->n_boxes; i++)
+    {
+      struct box *box = &boxplot->boxes[i];
+      struct statistic *statistic = &box->bw->parent.parent;
+      statistic->destroy (statistic);
+      free (box->label);
+    }
+  free (boxplot->boxes);
+  free (boxplot);
+}
+
+static const struct chart_class boxplot_chart_class =
+  {
+    boxplot_chart_draw,
+    boxplot_chart_destroy
+  };
diff --git a/src/output/charts/boxplot.h b/src/output/charts/boxplot.h
new file mode 100644 (file)
index 0000000..1e9d007
--- /dev/null
@@ -0,0 +1,27 @@
+/* PSPP - a program for statistical analysis.
+   Copyright (C) 2004, 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
+   the Free Software Foundation, either version 3 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, see <http://www.gnu.org/licenses/>. */
+
+#ifndef OUTPUT_CHARTS_BOXPLOT_H
+#define OUTPUT_CHARTS_BOXPLOT_H 1
+
+struct box_whisker;
+
+struct boxplot *boxplot_create (double y_min, double y_max, const char *title);
+void boxplot_add_box (struct boxplot *,
+                      struct box_whisker *, const char *label);
+struct chart *boxplot_get_chart (struct boxplot *);
+
+#endif /* output/charts/boxplot.h */