gruff 0.16.0 → 0.17.0

data/lib/gruff/helper/bar_value_label.rb

@@ -49,26 +49,4 @@ module Gruff::BarValueLabel
       yield x, left_y, val, metrics.width, metrics.height
     end
   end
-
-  # @private
-  class StackedBar
-    def initialize
-      @bars = []
-    end
-
-    def add(bar, index)
-      bars = @bars[index] || []
-      bars << bar
-      @bars[index] = bars
-    end
-
-    def prepare_rendering(format, proc_text_metrics, &block)
-      @bars.each do |bars|
-        value = bars.sum(&:value)
-        bar = bars.last
-        bar_value_label = bar.class.new(bar.coordinate, value)
-        bar_value_label.prepare_rendering(format, proc_text_metrics, &block)
-      end
-    end
-  end
 end

data/lib/gruff/helper/stacked_mixin.rb

@@ -21,4 +21,20 @@ module Gruff::Base::StackedMixin
 
     raise "Can't handle negative values in stacked graph" if minimum_value < 0
   end
+
+  def normalized_stacked_bars
+    @normalized_stacked_bars ||= begin
+      stacked_bars = Array.new(column_count) { [] }
+      store.norm_data.each_with_index do |data_row, row_index|
+        data_row.points.each_with_index do |data_point, point_index|
+          stacked_bars[point_index] << BarData.new(data_point, store.data[row_index].points[point_index], data_row.color)
+        end
+      end
+      stacked_bars
+    end
+  end
+
+  # @private
+  class BarData < Struct.new(:point, :value, :color)
+  end
 end

data/lib/gruff/histogram.rb

@@ -44,11 +44,15 @@ private
 
   def setup_data
     @data.each do |(name, data_points, color)|
-      bins, freqs = HistogramArray.new(data_points).histogram(bin_width: @bin_width, min: @minimum_bin, max: @maximum_bin)
-      bins.each_with_index do |bin, index|
-        @labels[index] = bin
+      if data_points.empty?
+        store.add(name, [], color)
+      else
+        bins, freqs = HistogramArray.new(data_points).histogram(bin_width: @bin_width, min: @minimum_bin, max: @maximum_bin)
+        bins.each_with_index do |bin, index|
+          @labels[index] = bin
+        end
+        store.add(name, freqs, color)
       end
-      store.add(name, freqs, color)
     end
 
     super

data/lib/gruff/line.rb

@@ -25,7 +25,7 @@ class Gruff::Line < Gruff::Base
   attr_writer :line_width
   attr_writer :dot_radius
 
-  # default is +'circle'+, other options include square.
+  # default is +'circle'+, other options include +square+ and +diamond+.
   attr_writer :dot_style
 
   # Hide parts of the graph to fit more data points, or for a different appearance.
@@ -273,10 +273,7 @@ private
     return unless @show_vertical_markers
 
     (0..@marker_x_count).each do |index|
-      x = @graph_left + @graph_width - (index * @graph_width / @marker_x_count)
-
-      Gruff::Renderer::Line.new(renderer, color: @marker_color).render(x, @graph_bottom, x, @graph_top)
-      Gruff::Renderer::Line.new(renderer, color: @marker_shadow_color).render(x + 1, @graph_bottom, x + 1, @graph_top) if @marker_shadow_color
+      draw_marker_vertical_line(@graph_left + @graph_width - (index * @graph_width / @marker_x_count))
     end
   end
 

data/lib/gruff/mini/legend.rb

@@ -21,7 +21,7 @@ module Gruff
 
         @legend_labels = store.data.map(&:label)
 
-        legend_height = scale_fontsize((store.length * calculate_line_height) + @top_margin + @bottom_margin)
+        legend_height = scale((store.length * calculate_line_height) + @top_margin + @bottom_margin)
 
         @original_rows = @raw_rows
         @original_columns = @raw_columns
@@ -32,7 +32,7 @@ module Gruff
           @columns += calculate_legend_width + @left_margin
         else
           font = @legend_font.dup
-          font.size = scale_fontsize(font.size)
+          font.size = scale(font.size)
           @rows += store.length * calculate_caps_height(font) * 1.7
         end
 
@@ -45,7 +45,7 @@ module Gruff
 
       def calculate_legend_width
         width = @legend_labels.map { |label| calculate_width(@legend_font, label) }.max
-        scale_fontsize(width + (40 * 1.7))
+        scale(width + (40 * 1.7))
       end
 
       ##
@@ -69,9 +69,9 @@ module Gruff
 
         @legend_labels.each_with_index do |legend_label, index|
           # Draw label
-          label = truncate_legend_label(legend_label)
-          text_renderer = Gruff::Renderer::Text.new(renderer, label, font: @legend_font)
           x_offset = current_x_offset + (legend_square_width * 1.7)
+          label = truncate_legend_label(legend_label, x_offset)
+          text_renderer = Gruff::Renderer::Text.new(renderer, label, font: @legend_font)
           text_renderer.add_to_render_queue(@raw_columns, 1.0, x_offset, current_y_offset, Magick::WestGravity)
 
           # Now draw box with color of this dataset
@@ -90,13 +90,13 @@ module Gruff
       #
       #   Department of Hu...
 
-      def truncate_legend_label(label)
+      def truncate_legend_label(label, x_offset)
         truncated_label = label.to_s
 
         font = @legend_font.dup
-        font.size = scale_fontsize(font.size)
-        max_width = @columns - @legend_left_margin - @right_margin
-        while calculate_width(font, truncated_label) > max_width && truncated_label.length > 1
+        font.size = scale(font.size)
+        max_width = @columns - scale(x_offset) - @right_margin
+        while calculate_width(font, "#{truncated_label}...") > max_width && truncated_label.length > 1
           truncated_label = truncated_label[0..truncated_label.length - 2]
         end
         truncated_label + (truncated_label.length < label.to_s.length ? '...' : '')

data/lib/gruff/net.rb

@@ -43,6 +43,21 @@ private
     @marker_font.bold = true
   end
 
+  def setup_drawing
+    @center_labels_over_point = false
+    super
+  end
+
+  def setup_graph_measurements
+    super
+
+    @radius = @graph_height / 2.0
+    @circle_radius = @dot_radius || clip_value_if_greater_than(@columns / (store.norm_data.first.points.size * 2.5), 5.0)
+    @stroke_width  = @line_width || clip_value_if_greater_than(@columns / (store.norm_data.first.points.size * 4.0), 5.0)
+    @center_x = @graph_left + (@graph_width / 2.0)
+    @center_y = @graph_top + (@graph_height / 2.0) + 10
+  end
+
   def draw_graph
     store.norm_data.each do |data_row|
       data_row.points.each_with_index do |data_point, index|
@@ -70,16 +85,6 @@ private
     end
   end
 
-  def setup_drawing
-    super
-
-    @radius = @graph_height / 2.0
-    @circle_radius = @dot_radius || clip_value_if_greater_than(@columns / (store.norm_data.first.points.size * 2.5), 5.0)
-    @stroke_width  = @line_width || clip_value_if_greater_than(@columns / (store.norm_data.first.points.size * 4), 5.0)
-    @center_x = @graph_left + (@graph_width / 2.0)
-    @center_y = @graph_top + (@graph_height / 2.0) + 10
-  end
-
   # the lines connecting in the center, with the first line vertical
   def draw_line_markers
     return if @hide_line_markers
@@ -102,6 +107,6 @@ private
     x = x_offset + ((radius + LABEL_MARGIN) * Math.sin(deg2rad(angle)))
     y = y_offset - ((radius + LABEL_MARGIN) * Math.cos(deg2rad(angle)))
 
-    draw_label_at(1.0, 1.0, x, y, amount, Magick::CenterGravity)
+    draw_label_at(1.0, 1.0, x, y, amount, gravity: Magick::CenterGravity)
   end
 end

data/lib/gruff/pie.rb

@@ -61,6 +61,11 @@ private
     @label_formatting = ->(value, percentage) { @show_values_as_labels ? value.to_s : "#{percentage}%" }
   end
 
+  def setup_drawing
+    @center_labels_over_point = false
+    super
+  end
+
   def draw_graph
     slices.each do |slice|
       if slice.value > 0
@@ -136,7 +141,7 @@ private
     if slice.percentage >= @hide_labels_less_than
       x, y = label_coordinates_for slice
       label = @label_formatting.call(slice.value, slice.percentage)
-      draw_label_at(1.0, 1.0, x, y, label, Magick::CenterGravity)
+      draw_label_at(1.0, 1.0, x, y, label, gravity: Magick::CenterGravity)
     end
   end
 

data/lib/gruff/renderer/dot.rb

@@ -11,31 +11,42 @@ module Gruff
       @opacity = opacity
     end
 
-    def render(new_x, new_y, circle_radius)
-      # @renderer.draw.push # TODO
+    def render(new_x, new_y, radius)
+      @renderer.draw.push
       @renderer.draw.stroke_width(@width)
       @renderer.draw.stroke(@color)
       @renderer.draw.fill(@color)
       @renderer.draw.fill_opacity(@opacity)
-      if @style.to_s == 'square'
-        square(new_x, new_y, circle_radius)
+      case @style.to_sym
+      when :square
+        square(new_x, new_y, radius)
+      when :diamond
+        diamond(new_x, new_y, radius)
       else
-        circle(new_x, new_y, circle_radius)
+        circle(new_x, new_y, radius)
       end
-      # @renderer.draw.pop # TODO
+      @renderer.draw.pop
     end
 
-    def circle(new_x, new_y, circle_radius)
-      @renderer.draw.circle(new_x, new_y, new_x - circle_radius, new_y)
+    def circle(new_x, new_y, radius)
+      @renderer.draw.circle(new_x, new_y, new_x - radius, new_y)
     end
 
-    def square(new_x, new_y, circle_radius)
-      offset = (circle_radius * 0.8).to_i
-      corner1 = new_x - offset
-      corner2 = new_y - offset
-      corner3 = new_x + offset
-      corner4 = new_y + offset
+    def square(new_x, new_y, radius)
+      corner1 = new_x - radius
+      corner2 = new_y - radius
+      corner3 = new_x + radius
+      corner4 = new_y + radius
       @renderer.draw.rectangle(corner1, corner2, corner3, corner4)
     end
+
+    def diamond(new_x, new_y, radius)
+      polygon = []
+      polygon += [new_x - radius, new_y]
+      polygon += [new_x, new_y + radius]
+      polygon += [new_x + radius, new_y]
+      polygon += [new_x, new_y - radius]
+      @renderer.draw.polygon(*polygon)
+    end
   end
 end

data/lib/gruff/renderer/renderer.rb

@@ -23,10 +23,6 @@ module Gruff
       end
     end
 
-    def background_image=(image)
-      @image = image
-    end
-
     def background(columns, rows, scale, theme_options)
       case theme_options[:background_colors]
       when Array

data/lib/gruff/renderer/text.rb

@@ -25,6 +25,7 @@ module Gruff
     end
 
     def render(width, height, x, y, gravity = Magick::NorthGravity)
+      @renderer.draw.push
       @renderer.draw.rotation = @rotation if @rotation
       @renderer.draw.fill = @font.color
       @renderer.draw.stroke = 'transparent'
@@ -37,9 +38,11 @@ module Gruff
                                      x, y,
                                      @text, @renderer.scale)
       @renderer.draw.rotation = -@rotation if @rotation
+      @renderer.draw.pop
     end
 
     def metrics
+      @renderer.draw.push
       @renderer.draw.font = @font.file_path
       @renderer.draw.font_weight = @font.weight
       @renderer.draw.pointsize = @font.size
@@ -50,7 +53,10 @@ module Gruff
       # So, in here, it just escape % in order to avoid SEGV.
       text = @text.to_s.gsub(/(%+)/) { ('%' * Regexp.last_match(1).size * 2).to_s }
 
-      @renderer.draw.get_type_metrics(@renderer.image, text)
+      metrics = @renderer.draw.get_type_metrics(@renderer.image, text)
+      @renderer.draw.pop
+
+      metrics
     end
   end
 end

data/lib/gruff/scatter.rb

@@ -114,20 +114,9 @@ private
     @x_label_margin = nil
   end
 
-  def draw_graph
-    store.norm_data.each do |data_row|
-      data_row.coordinates.each do |x_value, y_value|
-        next if y_value.nil? || x_value.nil?
-
-        new_x = get_x_coord(x_value, @graph_width, @graph_left)
-        new_y = @graph_top + (@graph_height - (y_value * @graph_height))
-
-        # Reset each time to avoid thin-line errors
-        stroke_width  = @stroke_width  || clip_value_if_greater_than(@columns / (store.norm_data.first[1].size * 4), 5.0)
-        circle_radius = @circle_radius || clip_value_if_greater_than(@columns / (store.norm_data.first[1].size * 2.5), 5.0)
-        Gruff::Renderer::Circle.new(renderer, color: data_row.color, width: stroke_width).render(new_x, new_y, new_x - circle_radius, new_y)
-      end
-    end
+  def setup_drawing
+    @center_labels_over_point = false
+    super
   end
 
   def setup_data
@@ -147,6 +136,22 @@ private
     super
   end
 
+  def draw_graph
+    store.norm_data.each do |data_row|
+      data_row.coordinates.each do |x_value, y_value|
+        next if y_value.nil? || x_value.nil?
+
+        new_x = get_x_coord(x_value, @graph_width, @graph_left)
+        new_y = @graph_top + (@graph_height - (y_value * @graph_height))
+
+        # Reset each time to avoid thin-line errors
+        stroke_width  = @stroke_width  || clip_value_if_greater_than(@columns / (store.norm_data.first[1].size * 4), 5.0)
+        circle_radius = @circle_radius || clip_value_if_greater_than(@columns / (store.norm_data.first[1].size * 2.5), 5.0)
+        Gruff::Renderer::Circle.new(renderer, color: data_row.color, width: stroke_width).render(new_x, new_y, new_x - circle_radius, new_y)
+      end
+    end
+  end
+
   def calculate_spread
     super
     @x_spread = @maximum_x_value.to_f - @minimum_x_value.to_f
@@ -170,10 +175,7 @@ private
     (0..marker_x_count).each do |index|
       # TODO: Fix the vertical lines, and enable them by default. Not pretty when they don't match up with top y-axis line
       if @show_vertical_markers
-        x = @graph_left + @graph_width - (index * increment_x_scaled)
-
-        Gruff::Renderer::Line.new(renderer, color: @marker_color).render(x, @graph_top, x, @graph_bottom)
-        Gruff::Renderer::Line.new(renderer, color: @marker_shadow_color).render(x, @graph_top + 1, x, @graph_bottom + 1) if @marker_shadow_color
+        draw_marker_vertical_line(@graph_left + @graph_width - (index * increment_x_scaled))
       end
 
       unless @hide_line_numbers

data/lib/gruff/side_bar.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'helper/bar_mixin'
+
 # Graph with individual horizontal bars instead of vertical bars.
 #
 # Here's how to set up a Gruff::SideBar.
@@ -19,6 +21,8 @@
 #   g.write('sidebar.png')
 #
 class Gruff::SideBar < Gruff::Base
+  include BarMixin
+
   # Spacing factor applied between bars.
   attr_writer :bar_spacing
 
@@ -79,8 +83,6 @@ private
     return if @hide_line_markers
 
     if @show_labels_for_bar_values
-      proc_text_metrics = ->(text) { text_metrics(@marker_font, text) }
-
       if maximum_value >= 0
         _, metrics = Gruff::BarValueLabel.metrics(maximum_value, @label_formatting, proc_text_metrics)
         @graph_right -= metrics.width
@@ -109,32 +111,33 @@ private
       minimum_value: minimum_value, maximum_value: maximum_value, spread: @spread
     )
 
-    proc_text_metrics = ->(text) { text_metrics(@marker_font, text) }
+    group_spacing = @group_spacing * @scale
+    group_left_y = @graph_top
 
-    store.norm_data.each_with_index do |data_row, row_index|
-      data_row.points.each_with_index do |data_point, point_index|
-        group_spacing = @group_spacing * @scale * point_index
-
-        left_y = @graph_top + (bars_width * point_index) + (bar_width * row_index) + padding + group_spacing
+    normalized_group_bars.each_with_index do |group_bars, group_index|
+      right_y = 0
+      group_bars.each_with_index do |bar, index|
+        left_y = group_left_y + (bar_width * index) + padding
         right_y = left_y + (bar_width * @bar_spacing)
 
-        left_x, right_x = conversion.get_top_bottom_scaled(data_point).sort
-
-        rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: data_row.color)
-        rect_renderer.render(left_x + AXIS_MARGIN, left_y, right_x + AXIS_MARGIN, right_y)
-
-        # Calculate center based on bar_width and current row
-        label_center = left_y + (bars_width / 2.0)
+        bottom_x, top_x = conversion.get_top_bottom_scaled(bar.point).sort
+        if bar.point != 0
+          rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: bar.color)
+          rect_renderer.render(bottom_x + AXIS_MARGIN, left_y, top_x, right_y)
+        end
 
-        # Subtract half a bar width to center left if requested
-        draw_label(label_center, point_index)
-        if @show_labels_for_bar_values
-          bar_value_label = Gruff::BarValueLabel::SideBar.new([left_x, left_y, right_x, right_y], store.data[row_index].points[point_index])
+        if @show_labels_for_bar_values && bar.value
+          bar_value_label = Gruff::BarValueLabel::SideBar.new([bottom_x, left_y, top_x, right_y], bar.value)
           bar_value_label.prepare_rendering(@label_formatting, proc_text_metrics) do |x, y, text, text_width, _text_height|
             draw_value_label(text_width, bar_width * @bar_spacing, x, y, text)
           end
         end
       end
+
+      label_center = group_left_y + (bars_width / 2.0)
+      draw_label(label_center, group_index)
+
+      group_left_y = right_y + padding + group_spacing
     end
   end
 
@@ -151,9 +154,7 @@ private
     (0..number_of_lines).each do |index|
       line_diff = (@graph_right - @graph_left) / number_of_lines
       x = @graph_right - (line_diff * index) - 1
-
-      Gruff::Renderer::Line.new(renderer, color: @marker_color).render(x, @graph_bottom, x, @graph_top)
-      Gruff::Renderer::Line.new(renderer, color: @marker_shadow_color).render(x, @graph_bottom + 1, x, @graph_top + 1) if @marker_shadow_color
+      draw_marker_vertical_line(x)
 
       unless @hide_line_numbers
         diff = index - number_of_lines
@@ -170,11 +171,15 @@ private
 
   def draw_label(y_offset, index)
     draw_unique_label(index) do
-      draw_label_at(@graph_left - LABEL_MARGIN, 1.0, 0.0, y_offset, @labels[index], Magick::EastGravity)
+      draw_label_at(@graph_left - LABEL_MARGIN, 1.0, 0.0, y_offset, @labels[index], gravity: Magick::EastGravity)
     end
   end
 
   def calculate_spacing
     @scale * @group_spacing * (column_count - 1)
   end
+
+  def proc_text_metrics
+    ->(text) { text_metrics(@marker_font, text) }
+  end
 end

data/lib/gruff/side_stacked_bar.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'helper/stacked_mixin'
+
 #
 # New gruff graph type added to enable sideways stacking bar charts
 # (basically looks like a x/y flip of a standard stacking bar chart)
@@ -66,50 +68,43 @@ private
     #
     # Columns sit stacked.
     bar_width = @graph_height / column_count
-    height = Array.new(column_count, 0)
-    length = Array.new(column_count, @graph_left)
     padding = (bar_width * (1 - @bar_spacing)) / 2
-    stack_bar_value_labels = Gruff::BarValueLabel::StackedBar.new
-
-    store.norm_data.each_with_index do |data_row, row_index|
-      data_row.points.each_with_index do |data_point, point_index|
-        ## using the original calculations from the stacked bar chart to get the difference between
-        ## part of the bart chart we wish to stack.
-        temp1 = @graph_left + (@graph_width - (data_point * @graph_width) - height[point_index])
-        temp2 = @graph_left + @graph_width - height[point_index]
-        difference = temp2 - temp1
-        difference = 0 if difference < 0
-
-        left_x = length[point_index]
-        left_y = @graph_top + (bar_width * point_index) + padding
-        right_x = left_x + difference
-        right_x -= @segment_spacing if row_index != store.columns - 1
-        right_y = left_y + (bar_width * @bar_spacing)
-        length[point_index] += difference
-        height[point_index] += (data_point * @graph_width)
-
-        bar_value_label = Gruff::BarValueLabel::SideBar.new([left_x, left_y, right_x, right_y], store.data[row_index].points[point_index])
-        stack_bar_value_labels.add(bar_value_label, point_index)
-
-        # if a data point is 0 it can result in weird really thing lines
-        # that shouldn't even be there being drawn on top of the existing
-        # bar - this is bad
-        if data_point != 0
-          rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: data_row.color)
-          rect_renderer.render(left_x, left_y, right_x, right_y)
-          # Calculate center based on bar_width and current row
-        end
-        # we still need to draw the labels
-        # Calculate center based on bar_width and current row
-        label_center = left_y + (bar_width / 2.0)
-        draw_label(label_center, point_index)
+
+    # Setup the BarConversion Object
+    conversion = Gruff::BarConversion.new(
+      top: @graph_right, bottom: @graph_left,
+      minimum_value: minimum_value, maximum_value: maximum_value, spread: @spread
+    )
+
+    proc_text_metrics = ->(text) { text_metrics(@marker_font, text) }
+
+    normalized_stacked_bars.each_with_index do |stacked_bars, stacked_index|
+      total = 0
+      left_y = @graph_top + (bar_width * stacked_index) + padding
+      right_y = left_y + (bar_width * @bar_spacing)
+
+      top_x = 0
+      stacked_bars.each do |bar|
+        next if bar.point == 0
+
+        bottom_x, = conversion.get_top_bottom_scaled(total)
+        bottom_x += @segment_spacing
+        top_x, = conversion.get_top_bottom_scaled(total + bar.point)
+
+        rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: bar.color)
+        rect_renderer.render(bottom_x, left_y, top_x, right_y)
+
+        total += bar.point
       end
-    end
 
-    if @show_labels_for_bar_values
-      proc_text_metrics = ->(text) { text_metrics(@marker_font, text) }
-      stack_bar_value_labels.prepare_rendering(@label_formatting, proc_text_metrics) do |x, y, text, text_width, _text_height|
-        draw_value_label(text_width, bar_width * @bar_spacing, x, y, text)
+      label_center = left_y + (bar_width / 2.0)
+      draw_label(label_center, stacked_index)
+
+      if @show_labels_for_bar_values
+        bar_value_label = Gruff::BarValueLabel::SideBar.new([@graph_left, left_y, top_x, right_y], stacked_bars.sum(&:value))
+        bar_value_label.prepare_rendering(@label_formatting, proc_text_metrics) do |x, y, text, text_width, _text_height|
+          draw_value_label(text_width, bar_width * @bar_spacing, x, y, text)
+        end
       end
     end
   end

data/lib/gruff/spider.rb

@@ -46,6 +46,11 @@ private
     @hide_line_markers.freeze
   end
 
+  def setup_drawing
+    @center_labels_over_point = false
+    super
+  end
+
   def setup_graph_measurements
     super
 
@@ -82,7 +87,7 @@ private
   end
 
   def normalize_points(value)
-    value * @unit_length
+    value.to_f * @unit_length
   end
 
   def draw_label(center_x, center_y, angle, radius, amount)
@@ -107,7 +112,7 @@ private
     x = x_offset + ((radius + r_offset) * Math.cos(angle))
     y = y_offset + ((radius + r_offset) * Math.sin(angle))
 
-    draw_label_at(metrics.width, metrics.height, x, y, amount, Magick::CenterGravity)
+    draw_label_at(metrics.width, metrics.height, x, y, amount, gravity: Magick::CenterGravity)
   end
 
   def draw_axes(center_x, center_y, radius, additive_angle, line_color = nil)

data/lib/gruff/stacked_area.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'helper/stacked_mixin'
+
 #
 # Here's how to set up a Gruff::StackedArea.
 #
@@ -35,9 +37,10 @@ private
 
     height = Array.new(column_count, 0)
 
-    data_points = nil
+    prev_data_points = nil
     store.norm_data.each do |data_row|
-      prev_data_points = data_points
+      next if data_row.points.empty?
+
       data_points = []
 
       data_row.points.each_with_index do |data_point, index|
@@ -69,6 +72,8 @@ private
       poly_points << data_points[1]
 
       Gruff::Renderer::Polygon.new(renderer, color: data_row.color).render(poly_points)
+
+      prev_data_points = data_points
     end
   end
 end