Start of area graphs
[imager-graph.git] / lib / Imager / Graph / Vertical.pm
index 89b3da891c46636076f37f27b7064b2fc1c43a9f..13b1a614afb5983c7e6df025667f6b3cfee85084 100644 (file)
@@ -79,6 +79,23 @@ sub add_line_data_series {
   return;
 }
 
+=item add_area_data_series(\@data, $series_name)
+
+Add a area data series to the graph.
+
+=cut
+
+sub add_area_data_series {
+  my $self = shift;
+  my $data_ref = shift;
+  my $series_name = shift;
+
+  $self->_add_data_series('area', $data_ref, $series_name);
+
+  return;
+}
+
+
 =item set_y_max($value)
 
 Sets the maximum y value to be displayed.  This will be ignored if the y_max is lower than the highest value.
@@ -99,6 +116,16 @@ sub set_y_min {
   $_[0]->{'custom_style'}->{'y_min'} = $_[1];
 }
 
+=item set_column_padding($int)
+
+Sets the padding between columns.  This is a percentage of the column width.  Defaults to 0.
+
+=cut
+
+sub set_column_padding {
+  $_[0]->{'custom_style'}->{'column_padding'} = $_[1];
+}
+
 =item set_range_padding($percentage)
 
 Sets the padding to be used, as a percentage.  For example, if your data ranges from 0 to 10, and you have a 20 percent padding, the y axis will go to 12.
@@ -144,31 +171,51 @@ sub draw {
   my @image_box = ( 0, 0, $img->getwidth-1, $img->getheight-1 );
   $self->_set_image_box(\@image_box);
 
+  my @chart_box = ( 0, 0, $img->getwidth-1, $img->getheight-1 );
+  $self->_draw_legend(\@chart_box);
+  if ($style->{title}{text}) {
+    $self->_draw_title($img, \@chart_box)
+      or return;
+  }
+
   # Scale the graph box down to the widest graph that can cleanly hold the # of columns.
-  $self->_get_data_range();
+  return unless $self->_get_data_range();
+  $self->_remove_tics_from_chart_box(\@chart_box);
   my $column_count = $self->_get_column_count();
 
   my $width = $self->_get_number('width');
   my $height = $self->_get_number('height');
-  my $size = $self->_get_number('size');
 
-  my $bottom = ($height - $size) / 2;
-  my $left   = ($width - $size) / 2;
+  my $graph_width = $chart_box[2] - $chart_box[0];
+  my $graph_height = $chart_box[3] - $chart_box[1];
+
+  my $col_width = ($graph_width - 1) / $column_count;
+  if ($col_width > 1) {
+    $graph_width = int($col_width) * $column_count + 1;
+  }
+  else {
+    $graph_width = $col_width * $column_count + 1;
+  }
+
+  my $tic_count = $self->_get_y_tics();
+  my $tic_distance = ($graph_height-1) / ($tic_count - 1);
+  $graph_height = int($tic_distance * ($tic_count - 1));
 
-  my $col_width = int($size / $column_count) -1;
-  my $graph_width = $col_width * $column_count + 1;
+  my $bottom = $chart_box[1];
+  my $left   = $chart_box[0];
 
-  my @graph_box = ( $left, $bottom, $left + $graph_width - 1, $bottom + $size - 1 );
-  $self->_set_graph_box(\@graph_box);
+  $self->{'_style'}{'graph_width'} = $graph_width;
+  $self->{'_style'}{'graph_height'} = $graph_height;
 
-  $self->_draw_legend();
+  my @graph_box = ($left, $bottom, $left + $graph_width, $bottom + $graph_height);
+  $self->_set_graph_box(\@graph_box);
 
   $img->box(
             color   => $self->_get_color('outline.line'),
             xmin    => $left,
             xmax    => $left+$graph_width,
             ymin    => $bottom,
-            ymax    => $bottom+$size,
+            ymax    => $bottom+$graph_height,
             );
 
   $img->box(
@@ -176,7 +223,7 @@ sub draw {
             xmin    => $left + 1,
             xmax    => $left+$graph_width - 1,
             ymin    => $bottom + 1,
-            ymax    => $bottom+$size -1 ,
+            ymax    => $bottom+$graph_height-1 ,
             filled  => 1,
             );
 
@@ -186,7 +233,7 @@ sub draw {
 
   my $zero_position;
   if ($value_range) {
-    $zero_position =  $bottom + $size - (-1*$min_value / $value_range) * ($size -1);
+    $zero_position =  $bottom + $graph_height - (-1*$min_value / $value_range) * ($graph_height-1);
   }
 
   if ($min_value < 0) {
@@ -195,7 +242,7 @@ sub draw {
             xmin    => $left + 1,
             xmax    => $left+$graph_width- 1,
             ymin    => $zero_position,
-            ymax    => $bottom+$size -1,
+            ymax    => $bottom+$graph_height - 1,
             filled  => 1,
     );
     $img->line(
@@ -208,14 +255,25 @@ sub draw {
   }
 
   if ($self->_get_data_series()->{'stacked_column'}) {
-    $self->_draw_stacked_columns();
+    return unless $self->_draw_stacked_columns();
   }
   if ($self->_get_data_series()->{'column'}) {
-    $self->_draw_columns();
+    return unless $self->_draw_columns();
   }
   if ($self->_get_data_series()->{'line'}) {
-    $self->_draw_lines();
+    return unless $self->_draw_lines();
+  }
+  if ($self->_get_data_series()->{'area'}) {
+    return unless $self->_draw_area();
+  }
+
+  if ($self->_get_y_tics()) {
+    $self->_draw_y_tics();
+  }
+  if ($self->_get_labels()) {
+    $self->_draw_x_tics();
   }
+
   return $self->_get_image();
 }
 
@@ -229,12 +287,13 @@ sub _get_data_range {
   my ($sc_min, $sc_max, $sc_cols) = $self->_get_stacked_column_range();
   my ($c_min, $c_max, $c_cols) = $self->_get_column_range();
   my ($l_min, $l_max, $l_cols) = $self->_get_line_range();
+  my ($a_min, $a_max, $a_cols) = $self->_get_area_range();
 
   # These are side by side...
   $sc_cols += $c_cols;
 
-  $min_value = $self->_min(STARTING_MIN_VALUE, $sc_min, $c_min, $l_min);
-  $max_value = $self->_max(0, $sc_max, $c_max, $l_max);
+  $min_value = $self->_min(STARTING_MIN_VALUE, $sc_min, $c_min, $l_min, $a_min);
+  $max_value = $self->_max(0, $sc_max, $c_max, $l_max, $a_max);
 
   my $config_min = $self->_get_number('y_min');
   my $config_max = $self->_get_number('y_max');
@@ -247,9 +306,6 @@ sub _get_data_range {
   }
 
   my $range_padding = $self->_get_number('range_padding');
-  if (!defined $range_padding) {
-    $range_padding = 10;
-  }
   if (defined $config_min) {
     $min_value = $config_min;
   }
@@ -277,11 +333,30 @@ sub _get_data_range {
       $max_value += $difference;
     }
   }
-  $column_count = $self->_max(0, $sc_cols, $l_cols);
+  $column_count = $self->_max(0, $sc_cols, $l_cols, $a_cols);
+
+  if ($self->_get_number('automatic_axis')) {
+    # In case this was set via a style, and not by the api method
+    eval { require Chart::Math::Axis; };
+    if ($@) {
+      return $self->_error("Can't use automatic_axis - $@");
+    }
+
+    my $axis = Chart::Math::Axis->new();
+    $axis->include_zero();
+    $axis->add_data($min_value, $max_value);
+    $max_value = $axis->top;
+    $min_value = $axis->bottom;
+    my $ticks     = $axis->ticks;
+    # The +1 is there because we have the bottom tick as well
+    $self->set_y_tics($ticks+1);
+  }
 
   $self->_set_max_value($max_value);
   $self->_set_min_value($min_value);
   $self->_set_column_count($column_count);
+
+  return 1;
 }
 
 sub _min {
@@ -332,6 +407,33 @@ sub _get_line_range {
   return ($min_value, $max_value, $column_count);
 }
 
+sub _get_area_range {
+  my $self = shift;
+  my $series = $self->_get_data_series()->{'area'};
+  return (undef, undef, 0) unless $series;
+
+  my $max_value = 0;
+  my $min_value = STARTING_MIN_VALUE;
+  my $column_count = 0;
+
+  my @series = @{$series};
+  foreach my $series (@series) {
+    my @data = @{$series->{'data'}};
+
+    if (scalar @data > $column_count) {
+      $column_count = scalar @data;
+    }
+
+    foreach my $value (@data) {
+      if ($value > $max_value) { $max_value = $value; }
+      if ($value < $min_value) { $min_value = $value; }
+    }
+  }
+
+  return ($min_value, $max_value, $column_count);
+}
+
+
 sub _get_column_range {
   my $self = shift;
 
@@ -397,6 +499,7 @@ sub _get_stacked_column_range {
 
 sub _draw_legend {
   my $self = shift;
+  my $chart_box = shift;
   my $style = $self->{'_style'};
 
   my @labels;
@@ -412,7 +515,7 @@ sub _draw_legend {
   }
 
   if ($style->{features}{legend} && (scalar @labels)) {
-    $self->SUPER::_draw_legend($self->_get_image(), \@labels, $self->_get_image_box())
+    $self->SUPER::_draw_legend($self->_get_image(), \@labels, $chart_box)
       or return;
   }
   return;
@@ -436,28 +539,24 @@ sub _draw_lines {
 
   my $width = $self->_get_number('width');
   my $height = $self->_get_number('height');
-  my $size = $self->_get_number('size');
 
-  my $bottom = ($height - $size) / 2;
-  my $left   = ($width - $size) / 2;
-
-  my $zero_position =  $bottom + $size - (-1*$min_value / $value_range) * ($size -1);
-
-  if ($self->_get_y_tics()) {
-    $self->_draw_y_tics();
-  }
-  if ($self->_get_labels()) {
-    $self->_draw_x_tics();
-  }
+  my $graph_width = $self->_get_number('graph_width');
+  my $graph_height = $self->_get_number('graph_height');
 
   my $line_series = $self->_get_data_series()->{'line'};
   my $series_counter = $self->_get_series_counter() || 0;
 
   my $has_columns = (defined $self->_get_data_series()->{'column'} || $self->_get_data_series->{'stacked_column'}) ? 1 : 0;
 
-  my $col_width = int($size / $column_count) -1;
-  my $graph_width = $col_width * $column_count + 1;
+  my $col_width = int($graph_width / $column_count) -1;
+
+  my $graph_box = $self->_get_graph_box();
+  my $left = $graph_box->[0] + 1;
+  my $bottom = $graph_box->[1];
 
+  my $zero_position =  $bottom + $graph_height - (-1*$min_value / $value_range) * ($graph_height - 1);
+
+  my $line_aa = $self->_get_number("lineaa");
   foreach my $series (@$line_series) {
     my @data = @{$series->{'data'}};
     my $data_size = scalar @data;
@@ -480,17 +579,17 @@ sub _draw_lines {
       $x1 += $has_columns * $interval / 2;
       $x2 += $has_columns * $interval / 2;
 
-      my $y1 = $bottom + ($value_range - $data[$i] + $min_value)/$value_range * $size;
-      my $y2 = $bottom + ($value_range - $data[$i + 1] + $min_value)/$value_range * $size;
+      my $y1 = $bottom + ($value_range - $data[$i] + $min_value)/$value_range * $graph_height;
+      my $y2 = $bottom + ($value_range - $data[$i + 1] + $min_value)/$value_range * $graph_height;
 
       push @marker_positions, [$x1, $y1];
-      $img->line(x1 => $x1, y1 => $y1, x2 => $x2, y2 => $y2, aa => 1, color => $color) || die $img->errstr;
+      $img->line(x1 => $x1, y1 => $y1, x2 => $x2, y2 => $y2, aa => $line_aa, color => $color) || die $img->errstr;
     }
 
     my $x2 = $left + ($data_size - 1) * $interval;
     $x2 += $has_columns * $interval / 2;
 
-    my $y2 = $bottom + ($value_range - $data[$data_size - 1] + $min_value)/$value_range * $size;
+    my $y2 = $bottom + ($value_range - $data[$data_size - 1] + $min_value)/$value_range * $graph_height;
 
     push @marker_positions, [$x2, $y2];
     foreach my $position (@marker_positions) {
@@ -500,9 +599,98 @@ sub _draw_lines {
   }
 
   $self->_set_series_counter($series_counter);
-  return;
+  return 1;
 }
 
+sub _draw_area {
+  my $self = shift;
+  my $style = $self->{'_style'};
+
+  my $img = $self->_get_image();
+
+  my $max_value = $self->_get_max_value();
+  my $min_value = $self->_get_min_value();
+  my $column_count = $self->_get_column_count();
+
+  my $value_range = $max_value - $min_value;
+
+  my $width = $self->_get_number('width');
+  my $height = $self->_get_number('height');
+
+  my $graph_width = $self->_get_number('graph_width');
+  my $graph_height = $self->_get_number('graph_height');
+
+  my $area_series = $self->_get_data_series()->{'area'};
+  my $series_counter = $self->_get_series_counter() || 0;
+
+  my $has_columns = (defined $self->_get_data_series()->{'column'} || $self->_get_data_series->{'stacked_column'}) ? 1 : 0;
+
+  my $col_width = int($graph_width / $column_count) -1;
+
+  my $graph_box = $self->_get_graph_box();
+  my $left = $graph_box->[0] + 1;
+  my $bottom = $graph_box->[1];
+  my $right = $graph_box->[2];
+  my $top = $graph_box->[3];
+
+  my $zero_position =  $bottom + $graph_height - (-1*$min_value / $value_range) * ($graph_height - 1);
+
+  my $line_aa = $self->_get_number("lineaa");
+  foreach my $series (@$area_series) {
+    my @data = @{$series->{'data'}};
+    my $data_size = scalar @data;
+
+    my $interval;
+    if ($has_columns) {
+      $interval = $graph_width / ($data_size);
+    }
+    else {
+      $interval = $graph_width / ($data_size - 1);
+    }
+    my $color = $self->_data_color($series_counter);
+
+    # We need to add these last, otherwise the next line segment will overwrite half of the marker
+    my @marker_positions;
+    my @polygon_points;
+    for (my $i = 0; $i < $data_size - 1; $i++) {
+      my $x1 = $left + $i * $interval;
+
+      $x1 += $has_columns * $interval / 2;
+
+      my $y1 = $bottom + ($value_range - $data[$i] + $min_value)/$value_range * $graph_height;
+
+      if ($i == 0) {
+        push @polygon_points, [$x1, $top];
+      }
+      push @polygon_points, [$x1, $y1];
+
+      push @marker_positions, [$x1, $y1];
+    }
+
+    my $x2 = $left + ($data_size - 1) * $interval;
+    $x2 += $has_columns * $interval / 2;
+
+    my $y2 = $bottom + ($value_range - $data[$data_size - 1] + $min_value)/$value_range * $graph_height;
+    push @polygon_points, [$x2, $y2];
+    push @polygon_points, [$x2, $top];
+    push @polygon_points, $polygon_points[0];
+
+    my @fill = $self->_data_fill($series_counter, [$left, $bottom, $right, $top]);
+    $img->polygon(points => [@polygon_points], @fill);
+
+    push @marker_positions, [$x2, $y2];
+    foreach my $position (@marker_positions) {
+      $self->_draw_line_marker($position->[0], $position->[1], $series_counter);
+    }
+    $series_counter++;
+  }
+
+  $self->_set_series_counter($series_counter);
+  return 1;
+}
+
+
+
 sub _line_marker {
   my ($self, $index) = @_;
 
@@ -527,9 +715,11 @@ sub _draw_line_marker {
   my $type = $style->{'shape'};
   my $radius = $style->{'radius'};
 
+  my $line_aa = $self->_get_number("lineaa");
+  my $fill_aa = $self->_get_number("fill.aa");
   if ($type eq 'circle') {
     my @fill = $self->_data_fill($series_counter, [$x1 - $radius, $y1 - $radius, $x1 + $radius, $y1 + $radius]);
-    $img->circle(x => $x1, y => $y1, r => $radius, aa => 1, filled => 1, @fill);
+    $img->circle(x => $x1, y => $y1, r => $radius, aa => $fill_aa, filled => 1, @fill);
   }
   elsif ($type eq 'square') {
     my @fill = $self->_data_fill($series_counter, [$x1 - $radius, $y1 - $radius, $x1 + $radius, $y1 + $radius]);
@@ -545,7 +735,7 @@ sub _draw_line_marker {
                     [$x1 + $radius, $y1],
                     [$x1, $y1 - $radius],
                   ],
-        filled => 1, color => $color, aa => 1);
+        filled => 1, color => $color, aa => $fill_aa);
   }
   elsif ($type eq 'triangle') {
     # The gradient really doesn't work for triangle
@@ -556,18 +746,18 @@ sub _draw_line_marker {
                     [$x1 + $radius, $y1 + $radius],
                     [$x1, $y1 - $radius],
                   ],
-        filled => 1, color => $color, aa => 1);
+        filled => 1, color => $color, aa => $fill_aa);
 
   }
   elsif ($type eq 'x') {
     my $color = $self->_data_color($series_counter);
-    $img->line(x1 => $x1 - $radius, y1 => $y1 -$radius, x2 => $x1 + $radius, y2 => $y1+$radius, aa => 1, color => $color) || die $img->errstr;
-    $img->line(x1 => $x1 + $radius, y1 => $y1 -$radius, x2 => $x1 - $radius, y2 => $y1+$radius, aa => 1, color => $color) || die $img->errstr;
+    $img->line(x1 => $x1 - $radius, y1 => $y1 -$radius, x2 => $x1 + $radius, y2 => $y1+$radius, aa => $line_aa, color => $color) || die $img->errstr;
+    $img->line(x1 => $x1 + $radius, y1 => $y1 -$radius, x2 => $x1 - $radius, y2 => $y1+$radius, aa => $line_aa, color => $color) || die $img->errstr;
   }
   elsif ($type eq 'plus') {
     my $color = $self->_data_color($series_counter);
-    $img->line(x1 => $x1, y1 => $y1 -$radius, x2 => $x1, y2 => $y1+$radius, aa => 1, color => $color) || die $img->errstr;
-    $img->line(x1 => $x1 + $radius, y1 => $y1, x2 => $x1 - $radius, y2 => $y1, aa => 1, color => $color) || die $img->errstr;
+    $img->line(x1 => $x1, y1 => $y1 -$radius, x2 => $x1, y2 => $y1+$radius, aa => $line_aa, color => $color) || die $img->errstr;
+    $img->line(x1 => $x1 + $radius, y1 => $y1, x2 => $x1 - $radius, y2 => $y1, aa => $line_aa, color => $color) || die $img->errstr;
   }
 }
 
@@ -585,21 +775,17 @@ sub _draw_columns {
 
   my $width = $self->_get_number('width');
   my $height = $self->_get_number('height');
-  my $size = $self->_get_number('size');
 
-  my $bottom = ($height - $size) / 2;
-  my $left   = ($width - $size) / 2 + 1;
+  my $graph_width = $self->_get_number('graph_width');
+  my $graph_height = $self->_get_number('graph_height');
 
-  my $zero_position =  int($bottom + $size - (-1*$min_value / $value_range) * ($size -1));
 
-  if ($self->_get_y_tics()) {
-    $self->_draw_y_tics();
-  }
-  if ($self->_get_labels()) {
-    $self->_draw_x_tics();
-  }
+  my $graph_box = $self->_get_graph_box();
+  my $left = $graph_box->[0] + 1;
+  my $bottom = $graph_box->[1];
+  my $zero_position =  int($bottom + $graph_height - (-1*$min_value / $value_range) * ($graph_height -1));
 
-  my $bar_width = int(($size)/ $column_count - 2);
+  my $bar_width = $graph_width / $column_count;
 
   my $outline_color;
   if ($style->{'features'}{'outline'}) {
@@ -608,6 +794,8 @@ sub _draw_columns {
 
   my $series_counter = $self->_get_series_counter() || 0;
   my $col_series = $self->_get_data_series()->{'column'};
+  my $column_padding_percent = $self->_get_number('column_padding') || 0;
+  my $column_padding = int($column_padding_percent * $bar_width / 100);
 
   # This tracks the series we're in relative to the starting series - this way colors stay accurate, but the columns don't start out too far to the right.
   my $column_series = 0;
@@ -619,27 +807,35 @@ sub _draw_columns {
     my $series = $col_series->[$series_pos];
     my @data = @{$series->{'data'}};
     my $data_size = scalar @data;
-    my $color = $self->_data_color($series_counter);
     for (my $i = 0; $i < $data_size; $i++) {
-      my $x1 = int($left + $bar_width * (scalar @$col_series * $i + $series_pos)) + scalar @$col_series * $i + $series_pos;
+      my $part1 = $bar_width * (scalar @$col_series * $i);
+      my $part2 = ($series_pos) * $bar_width;
+      my $x1 = $left + $part1 + $part2;
       if ($has_stacked_columns) {
-        $x1 += ($i + 1) * $bar_width + $i + 1;
+        $x1 += ($bar_width * ($i+1));
+      }
+      $x1 = int($x1);
+
+      my $x2 = int($x1 + $bar_width - $column_padding)-1;
+      # Special case for when bar_width is less than 1.
+      if ($x2 < $x1) {
+        $x2 = $x1;
       }
-      my $x2 = $x1 + $bar_width;
 
-      my $y1 = int($bottom + ($value_range - $data[$i] + $min_value)/$value_range * $size);
+      my $y1 = int($bottom + ($value_range - $data[$i] + $min_value)/$value_range * $graph_height);
 
       my $color = $self->_data_color($series_counter);
 
-    #  my @fill = $self->_data_fill($series_counter, [$x1, $y1, $x2, $zero_position]);
       if ($data[$i] > 0) {
-        $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position-1, color => $color, filled => 1);
+        my @fill = $self->_data_fill($series_counter, [$x1, $y1, $x2, $zero_position-1]);
+        $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position-1, @fill);
         if ($style->{'features'}{'outline'}) {
           $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position, color => $outline_color);
         }
       }
       else {
-        $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1, color => $color, filled => 1);
+        my @fill = $self->_data_fill($series_counter, [$x1, $zero_position+1, $x2, $y1]);
+        $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1, @fill);
         if ($style->{'features'}{'outline'}) {
           $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1+1, color => $outline_color);
         }
@@ -650,7 +846,7 @@ sub _draw_columns {
     $column_series++;
   }
   $self->_set_series_counter($series_counter);
-  return;
+  return 1;
 }
 
 sub _draw_stacked_columns {
@@ -667,49 +863,60 @@ sub _draw_stacked_columns {
   my $graph_box = $self->_get_graph_box();
   my $left = $graph_box->[0] + 1;
   my $bottom = $graph_box->[1];
-  my $size = $self->_get_number('size');
 
-  if ($self->_get_y_tics()) {
-    $self->_draw_y_tics();
-  }
-  if ($self->_get_labels()) {
-    $self->_draw_x_tics();
-  }
+  my $graph_width = $self->_get_number('graph_width');
+  my $graph_height = $self->_get_number('graph_height');
 
-  my $bar_width = int($size / $column_count -2);
+  my $bar_width = $graph_width / $column_count;
   my $column_series = 0;
   if (my $column_series_data = $self->_get_data_series()->{'column'}) {
     $column_series = (scalar @$column_series_data);
   }
   $column_series++;
 
+  my $column_padding_percent = $self->_get_number('column_padding') || 0;
+  if ($column_padding_percent < 0) {
+    return $self->_error("Column padding less than 0");
+  }
+  if ($column_padding_percent > 100) {
+    return $self->_error("Column padding greater than 0");
+  }
+  my $column_padding = int($column_padding_percent * $bar_width / 100);
+
   my $outline_color;
   if ($style->{'features'}{'outline'}) {
     $outline_color = $self->_get_color('outline.line');
   }
 
-  my $zero_position =  $bottom + $size - (-1*$min_value / $value_range) * ($size -1);
+  my $zero_position =  $bottom + $graph_height - (-1*$min_value / $value_range) * ($graph_height -1);
   my $col_series = $self->_get_data_series()->{'stacked_column'};
   my $series_counter = $self->_get_series_counter() || 0;
+
   foreach my $series (@$col_series) {
     my @data = @{$series->{'data'}};
     my $data_size = scalar @data;
-    my $color = $self->_data_color($series_counter);
     for (my $i = 0; $i < $data_size; $i++) {
-      my $x1 = int($left + $bar_width * ($column_series * $i)) + $column_series * $i;
-#      my $x1 = $left + $i * $size / ($data_size);
-      my $x2 = $x1 + $bar_width;
+      my $part1 = $bar_width * $i * $column_series;
+      my $part2 = 0;
+      my $x1 = int($left + $part1 + $part2);
+      my $x2 = int($x1 + $bar_width - $column_padding) - 1;
+      # Special case for when bar_width is less than 1.
+      if ($x2 < $x1) {
+        $x2 = $x1;
+      }
 
-      my $y1 = $bottom + ($value_range - $data[$i] + $min_value)/$value_range * $size;
+      my $y1 = int($bottom + ($value_range - $data[$i] + $min_value)/$value_range * $graph_height);
 
       if ($data[$i] > 0) {
-        $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position-1, color => $color, filled => 1);
+        my @fill = $self->_data_fill($series_counter, [$x1, $y1, $x2, $zero_position-1]);
+        $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position-1, @fill);
         if ($style->{'features'}{'outline'}) {
           $img->box(xmin => $x1, xmax => $x2, ymin => $y1, ymax => $zero_position, color => $outline_color);
         }
       }
       else {
-        $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1, color => $color, filled => 1);
+        my @fill = $self->_data_fill($series_counter, [$x1, $zero_position+1, $x2, $y1]);
+        $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1, @fill);
         if ($style->{'features'}{'outline'}) {
           $img->box(xmin => $x1, xmax => $x2, ymin => $zero_position+1, ymax => $y1+1, color => $outline_color);
         }
@@ -719,7 +926,7 @@ sub _draw_stacked_columns {
     $series_counter++;
   }
   $self->_set_series_counter($series_counter);
-  return;
+  return 1;
 }
 
 sub _add_data_series {
@@ -742,6 +949,32 @@ sub _add_data_series {
 
 =over
 
+=item show_horizontal_gridlines()
+
+Shows horizontal gridlines at the y-tics.
+
+=cut
+
+sub show_horizontal_gridlines {
+    $_[0]->{'custom_style'}->{'horizontal_gridlines'} = 1;
+}
+
+=item use_automatic_axis()
+
+Automatically scale the Y axis, based on L<Chart::Math::Axis>.  If Chart::Math::Axis isn't installed, this sets an error and returns undef.  Returns 1 if it is installed.
+
+=cut
+
+sub use_automatic_axis {
+  eval { require Chart::Math::Axis; };
+  if ($@) {
+    return $_[0]->_error("use_automatic_axis - $@\nCalled from ".join(' ', caller)."\n");
+  }
+  $_[0]->{'custom_style'}->{'automatic_axis'} = 1;
+  return 1;
+}
+
+
 =item set_y_tics($count)
 
 Set the number of Y tics to use.  Their value and position will be determined by the data range.
@@ -753,7 +986,109 @@ sub set_y_tics {
 }
 
 sub _get_y_tics {
-  return $_[0]->{'y_tics'};
+  return $_[0]->{'y_tics'} || 0;
+}
+
+sub _remove_tics_from_chart_box {
+  my $self = shift;
+  my $chart_box = shift;
+
+  # XXX - bad default
+  my $tic_width = $self->_get_y_tic_width() || 10;
+  my @y_tic_box = ($chart_box->[0], $chart_box->[1], $chart_box->[0] + $tic_width, $chart_box->[3]);
+
+  # XXX - bad default
+  my $tic_height = $self->_get_x_tic_height() || 10;
+  my @x_tic_box = ($chart_box->[0], $chart_box->[3] - $tic_height, $chart_box->[2], $chart_box->[3]);
+
+  $self->_remove_box($chart_box, \@y_tic_box);
+  $self->_remove_box($chart_box, \@x_tic_box);
+
+  # If there's no title, the y-tics will be part off-screen.  Half of the x-tic height should be more than sufficient.
+  my @y_tic_tops = ($chart_box->[0], $chart_box->[1], $chart_box->[2], $chart_box->[1] + int($tic_height / 2));
+  $self->_remove_box($chart_box, \@y_tic_tops);
+
+  # Make sure that the first and last label fit
+  if (my $labels = $self->_get_labels()) {
+    if (my @box = $self->_text_bbox($labels->[0], 'legend')) {
+      my @remove_box = ($chart_box->[0],
+                        $chart_box->[1],
+                        $chart_box->[0] + int($box[2] / 2) + 1,
+                        $chart_box->[3]
+                        );
+
+      $self->_remove_box($chart_box, \@remove_box);
+    }
+    if (my @box = $self->_text_bbox($labels->[-1], 'legend')) {
+      my @remove_box = ($chart_box->[2] - int($box[2] / 2) - 1,
+                        $chart_box->[1],
+                        $chart_box->[2],
+                        $chart_box->[3]
+                        );
+
+      $self->_remove_box($chart_box, \@remove_box);
+    }
+  }
+}
+
+sub _get_y_tic_width{
+  my $self = shift;
+  my $min = $self->_get_min_value();
+  my $max = $self->_get_max_value();
+  my $tic_count = $self->_get_y_tics();
+
+  my $interval = ($max - $min) / ($tic_count - 1);
+
+  my %text_info = $self->_text_style('legend')
+    or return;
+
+  my $max_width = 0;
+  for my $count (0 .. $tic_count - 1) {
+    my $value = sprintf("%.2f", ($count*$interval)+$min);
+
+    my @box = $self->_text_bbox($value, 'legend');
+    my $width = $box[2] - $box[0];
+
+    # For the tic width
+    $width += 10;
+    if ($width > $max_width) {
+      $max_width = $width;
+    }
+  }
+
+  return $max_width;
+}
+
+sub _get_x_tic_height {
+  my $self = shift;
+
+  my $labels = $self->_get_labels();
+
+  if (!$labels) {
+        return;
+  }
+
+  my $tic_count = (scalar @$labels) - 1;
+
+  my %text_info = $self->_text_style('legend')
+    or return;
+
+  my $max_height = 0;
+  for my $count (0 .. $tic_count) {
+    my $label = $labels->[$count];
+
+    my @box = $self->_text_bbox($label, 'legend');
+
+    my $height = $box[3] - $box[1];
+
+    # Padding + the tic
+    $height += 10;
+    if ($height > $max_height) {
+      $max_height = $height;
+    }
+  }
+
+  return $max_height;
 }
 
 sub _draw_y_tics {
@@ -771,18 +1106,23 @@ sub _draw_y_tics {
   my %text_info = $self->_text_style('legend')
     or return;
 
+  my $line_style = $self->_get_color('outline.line');
+  my $show_gridlines = $self->_get_number('horizontal_gridlines');
   my $tic_distance = ($graph_box->[3] - $graph_box->[1]) / ($tic_count - 1);
   for my $count (0 .. $tic_count - 1) {
     my $x1 = $graph_box->[0] - 5;
     my $x2 = $graph_box->[0] + 5;
-    my $y1 = $graph_box->[3] - ($count * $tic_distance) + 1;
+    my $y1 = int($graph_box->[3] - ($count * $tic_distance));
 
-    my $value = sprintf("%.2f", ($count*$interval)+$min);
+    my $value = ($count*$interval)+$min;
+    if ($interval < 1 || ($value != int($value))) {
+        $value = sprintf("%.2f", $value);
+    }
 
     my @box = $self->_text_bbox($value, 'legend')
       or return;
 
-    $img->line(x1 => $x1, x2 => $x2, y1 => $y1, y2 => $y1, aa => 1, color => '000000');
+    $img->line(x1 => $x1, x2 => $x2, y1 => $y1, y2 => $y1, aa => 1, color => $line_style);
 
     my $width = $box[2];
     my $height = $box[3];
@@ -792,6 +1132,16 @@ sub _draw_y_tics {
                  y    => ($y1 + ($height / 2)),
                  text => $value
                 );
+
+    if ($show_gridlines) {
+      # XXX - line styles!
+      for (my $i = $graph_box->[0]; $i < $graph_box->[2]; $i += 6) {
+        my $x1 = $i;
+        my $x2 = $i + 2;
+        if ($x2 > $graph_box->[2]) { $x2 = $graph_box->[2]; }
+        $img->line(x1 => $x1, x2 => $x2, y1 => $y1, y2 => $y1, aa => 1, color => $line_style);
+      }
+    }
   }
 
 }
@@ -818,17 +1168,40 @@ sub _draw_x_tics {
   my %text_info = $self->_text_style('legend')
     or return;
 
+  # If automatic axis is turned on, let's be selective about what labels we draw.
+  my $max_size = 0;
+  my $tic_skip = 0;
+  if ($self->_get_number('automatic_axis')) {
+    foreach my $label (@$labels) {
+      my @box = $self->_text_bbox($label, 'legend');
+      if ($box[2] > $max_size) {
+        $max_size = $box[2];
+      }
+    }
+
+    # Give the max_size some padding...
+    $max_size *= 1.2;
+
+    $tic_skip = int($max_size / $tic_distance) + 1;
+  }
+
+  my $line_style = $self->_get_color('outline.line');
+
   for my $count (0 .. $tic_count) {
+    next if ($count % ($tic_skip + 1));
     my $label = $labels->[$count];
     my $x1 = $graph_box->[0] + ($tic_distance * $count);
 
     if ($has_columns) {
       $x1 += $tic_distance / 2;
     }
+
+    $x1 = int($x1);
+
     my $y1 = $graph_box->[3] + 5;
     my $y2 = $graph_box->[3] - 5;
 
-    $img->line(x1 => $x1, x2 => $x1, y1 => $y1, y2 => $y2, aa => 1, color => '000000');
+    $img->line(x1 => $x1, x2 => $x1, y1 => $y1, y2 => $y2, aa => 1, color => $line_style);
 
     my @box = $self->_text_bbox($label, 'legend')
       or return;