gruff 0.14.0-java → 0.17.0-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 49e695ac5551cf4ec68c09c9d272219433df913f0871cf108373fabffc73492f
-  data.tar.gz: 7724a089b1d53c511511301cb184af87a2fbe6ce92a9836fc053df5e7c39005a
+  metadata.gz: 1bee4569b7c7780fa68a3ae0273b921bf4e374ec207cd6381cd7feeb0ae8fd22
+  data.tar.gz: 3ceff033fcb2543ea8852fb11c4d56a5a1cf7dbd9dbf1bb62974602edd11b974
 SHA512:
-  metadata.gz: ba0f075b7a210e181b7a503bc1701cd43b618ba043dc44d2ef9c24d7db212687a840b8e92083c9e92d9d69d50f83d34d7b02ea4d2d0c33d1705abca2a18566d9
-  data.tar.gz: ce4e553a52dd6a1c85bf5fbcca5f37bebe4c9961627c213f96c10be08e611551a7c2927f3422a733208fd33363cd2767b8b0eab042d9de4707e2b4e8d82644d4
+  metadata.gz: c4e5834e0d3efa6bf619591f009d4e63d22c26e0e3223bdb7861662a89ae2985f62d98141584d3c493fc29ad13af8597bb36466a5df956ae2feb159c4179ca6b
+  data.tar.gz: 80599d2b8eb034218d9af18fbc7f7c01ba2c66d7b17a16de36a9417171376c6fadf794e69011b34ec777c249604d1cfb48087eee97f3e6cdb1c0f84ad51bc22c

data/.github/workflows/ci.yml

@@ -6,55 +6,71 @@ on:
   pull_request:
     branches: [ master ]
 
+permissions:
+  contents: read
+
 jobs:
   lint:
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-20.04
     timeout-minutes: 20
     name: Lint
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
       with:
         ref: ${{ github.event.pull_request.head.sha }}
-    - name: Set up Ruby 2.4
+    - name: Set up Ruby 2.5
       uses: ruby/setup-ruby@master
       with:
-        ruby-version: '2.4'
+        ruby-version: '2.5'
         bundler-cache: true
     - name: Install dependencies
       run: bundle install --path=vendor/bundle --jobs 4 --retry 3
     - name: RuboCop Problem Matchers
-      uses: r7kamura/rubocop-problem-matchers-action@v1.1.0
+      uses: r7kamura/rubocop-problem-matchers-action@v1
     - name: Run tests
       run: bundle exec rubocop
 
   test-ruby:
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-20.04
     timeout-minutes: 20
     strategy:
       matrix:
-        ruby-version: ['2.4', '2.5', '2.6', '2.7', '3.0']
+        ruby-version: ['2.5', '2.6', '2.7', '3.0', '3.1']
+        imagemagick-version:
+          - { full: 7.1.0-35, major-minor: '7.0' }
     name: Ruby ${{ matrix.ruby-version }}
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
+    - name: Cache ImageMagick
+      uses: actions/cache@v3
+      with:
+        path: ./build-ImageMagick
+        key: v1-linux-imagemagick-${{ matrix.imagemagick-version.full }}
+        restore-keys: |
+          v1-linux-imagemagick-${{ matrix.imagemagick-version.full }}
+    - name: Install ImageMagick ${{ matrix.imagemagick-version.full }}
+      run: |
+        export IMAGEMAGICK_VERSION=${{ matrix.imagemagick-version.full }}
+        ./before_install_linux.sh
     - name: Set up Ruby
       uses: ruby/setup-ruby@master
       with:
         ruby-version: ${{ matrix.ruby-version }}
-        bundler-cache: true
+        bundler-cache: false
     - name: Install dependencies
       run: bundle install --path=vendor/bundle --jobs 4 --retry 3
     - name: Run tests
       run: bundle exec rake
 
   test-jruby:
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-20.04
     timeout-minutes: 20
     strategy:
       matrix:
-        ruby-version: ['9.2.19.0']
+        ruby-version: ['9.2.20.1', '9.3.4.0']
     name: JRuby ${{ matrix.ruby-version }}
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Set up Ruby
       uses: ruby/setup-ruby@master
       with:

data/.gitignore

@@ -1,6 +1,7 @@
 *~
 /.idea/
 /pkg/
+/doc/
 /coverage/
 /vendor/
 /test/output/

data/.rubocop.yml

@@ -1,15 +1,17 @@
-inherit_from: .rubocop_todo.yml
 require:
   - rubocop-performance
   - rubocop-rake
 
 AllCops:
     EnabledByDefault: true
-    TargetRubyVersion: 2.4
+    TargetRubyVersion: 2.5
     Exclude:
         - 'vendor/bundle/**/*'
         - 'rails_generators/gruff/**/*'
 
+Metrics:
+  Enabled: false
+
 Gemspec/OrderedDependencies:
   Enabled: false
 
@@ -19,9 +21,6 @@ Gemspec/RequiredRubyVersion:
 Layout/AccessModifierIndentation:
   EnforcedStyle: outdent
 
-Layout/ClassStructure:
-  Enabled: false
-
 Layout/FirstMethodArgumentLineBreak:
   Enabled: false
 
@@ -34,25 +33,31 @@ Layout/MultilineMethodArgumentLineBreaks:
 Layout/MultilineAssignmentLayout:
   EnforcedStyle: same_line
 
-Lint/AssignmentInCondition:
+Layout/RedundantLineBreak:
+  Enabled: false
+
+Layout/SingleLineBlockChain:
   Enabled: false
 
 Lint/ConstantResolution:
   Enabled: false
 
+Lint/FloatComparison:
+  Enabled: false
+
 Lint/NumberConversion:
   Enabled: false
 
 Naming/MethodParameterName:
   Enabled: false
 
-Style/AccessorGrouping:
+Naming/VariableNumber:
   Enabled: false
 
-Style/ClassAndModuleChildren:
+Style/AccessorGrouping:
   Enabled: false
 
-Style/ConditionalAssignment:
+Style/ClassAndModuleChildren:
   Enabled: false
 
 Style/ConstantVisibility:
@@ -64,12 +69,15 @@ Style/Copyright:
 Style/Documentation:
   Enabled: true
 
-Style/FormatString:
-  EnforcedStyle: sprintf
+Style/DocumentationMethod:
+  Enabled: false
 
-Style/FormatStringToken:
+Style/DocumentDynamicEvalDefinition:
   Enabled: false
 
+Style/FormatString:
+  EnforcedStyle: sprintf
+
 Style/GuardClause:
   Enabled: false
 
@@ -94,9 +102,6 @@ Style/MissingElse:
 Style/Next:
   Enabled: false
 
-Style/NumericLiterals:
-  Enabled: false
-
 Style/NumericPredicate:
   Enabled: false
 
@@ -109,17 +114,8 @@ Style/RedundantBegin:
 Style/SingleLineBlockParams:
   Enabled: false
 
-Style/SpecialGlobalVars:
-  Enabled: false
-
 Style/StructInheritance:
   Enabled: false
 
-Style/SymbolArray:
-  Enabled: false
-
-Style/TernaryParentheses:
-  Enabled: false
-
 Performance/ChainArrayAllocation:
   Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,57 @@
 # Change Log
 
+## 0.17.0
+- Add Gruff::Base#label_rotation= method to rotate bottom label (#599)
+- Mark #label_stagger_height= as deprecated (#598)
+- Fixed truncation of long legends in mini graph (#597)
+- Fix error when input empty data in Gruff::Bezier (#596)
+- Fix error when input empty data in Gruff::StackedArea (#595)
+- Fix error when input empty data in Gruff::BoxPlot (#594)
+- Fix error when input empty data in Gruff::Spider (#593)
+- Fix error when input empty data in Gruff::Histogram (#592)
+- Fix error when input empty data in Gruff::Area (#591)
+- Stop adjusting the square radius in Gruff::Line (#589)
+- Add diamond dot stype in Gruff::Line (#588)
+- Allow title to be set as an array (#587)
+- Allow labels to be set as an array corresponding to the data values (#586)
+- Fixed a bug that breaks the colors in the theme when using the add_color method (#583)
+- Ensure to raise an exception if use sort feature in Candlestick
+- Fix column_count in BoxPlot/Candlestick (#581)
+- Fix left/right graph margin (#579)
+
+## 0.16.0
+- Add Candlestick (#575)
+- Add BoxPlot (#574)
+- Raise exception if less data set was given in Gruff::Spider (#573)
+- Adjust default label offset value in Gruff::Pie
+- Remove has_left_labels from attribute (#571)
+- Remove center_labels_over_point from attribute (#570)
+- Adjust legend label / box position
+- Fix right margin when truncate the label
+- Adjust axis label position
+- Adjust left label margin
+- Fix axis label position with hide_line_markers
+- Fix margin of value label in Bar/StackedBar (#568)
+- Fix label position in Gruff::Spider (#567)
+- Fix axis label position with legend_at_bottom
+- Fix axis label position with hide_line_markers
+- Refactor: Remove instance variable usage
+- Remove Gruff::Scene (#566)
+- Fix exception with zero value in StackBar (#565)
+- Fix segment spacing in SideStackedBar (#564)
+- Fix margin of value label in SideBar/SideStackedBar (#563)
+- Raise exception if negative values were given in stacked graph (#562)
+- Fix centering in value label position (#561)
+- Fix duck typing support in Gruff::Histogram (#560)
+- Fix start drawing position to top in Pie and enabled sort option by default (#559)
+
+## 0.15.0
+- Fix SideStackedBar which bars overlap on the coordinate axes if data contains 0 (#558)
+- Lazy loading library to reduce memory usage (#556)
+- Drop Ruby 2.4 support (#554)
+- Fix sort drawing in Gruff::Line (#553)
+- Fix color value handling in Line#dataxy method (#552)
+
 ## 0.14.0
 - Update rails template (#547)
 - Remove singleton in order to support multi-thread processing (#546)

data/README.md

@@ -126,6 +126,13 @@ In progress!
 
 ![Histogram chart](https://raw.github.com/topfunky/gruff/master/test/expected/histogram.png)
 
+### Box plot
+
+![Box plot](https://raw.github.com/topfunky/gruff/master/test/expected/box_plot.png)
+
+### Candlestick
+
+![Candlestick](https://raw.github.com/topfunky/gruff/master/test/expected/candlestick.png)
 
 ## Documentation
 
@@ -133,7 +140,7 @@ http://www.rubydoc.info/github/topfunky/gruff/frames
 
 ## Supported Ruby Versions
 
-- Ruby 2.4 or later
+- Ruby 2.5 or later
 - JRuby 9.2.x or later
 
 ## Development
@@ -149,13 +156,13 @@ $ ./docker-launch.sh
 
 3. Run tests
 ```sh
-$ rake
+$ bundle exec rake
 ```
 
 If you have made changes that involve updating the expected image, you need to update the image with the following command after running tests.
 
 ```sh
-$ rake test:image:update
+$ bundle exec rake test:image:update
 ```
 
 ## Contributing

data/gruff.gemspec

@@ -11,14 +11,11 @@ Gem::Specification.new do |s|
   s.authors = ['Geoffrey Grosenbach', 'Uwe Kubosch']
   s.description = 'Beautiful graphs for one or multiple datasets. Can be used on websites or in documents.'
   s.email = 'boss@topfunky.com'
-  s.files = `git ls-files`.split.reject do |f|
-    f =~ /^test|^docker|^Rakefile/i
-  end
+  s.files = `git ls-files`.split.grep_v(/^test|^docker|^before_install_linux.sh|^Rakefile/i)
   s.homepage = 'https://github.com/topfunky/gruff'
   s.require_paths = %w[lib]
   s.summary = 'Beautiful graphs for one or multiple datasets.'
   s.license = 'MIT'
-  s.test_files = s.files.grep(%r{^(test|spec|features)/})
   s.executables = s.files.grep(%r{^bin/}).map { |f| File.basename(f) }
   s.specification_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
 
@@ -26,16 +23,18 @@ Gem::Specification.new do |s|
     s.platform = 'java'
     s.add_dependency 'rmagick4j'
   else
-    s.add_dependency 'rmagick'
-    s.add_development_dependency 'rubocop', '~> 1.12.1'
-    s.add_development_dependency 'rubocop-performance', '~> 1.10.2'
-    s.add_development_dependency 'rubocop-rake', '~> 0.5.1'
+    s.add_dependency 'rmagick', '>= 4.2'
+    s.add_development_dependency 'rubocop', '~> 1.28.2'
+    s.add_development_dependency 'rubocop-performance', '~> 1.13.3'
+    s.add_development_dependency 'rubocop-rake', '~> 0.6.0'
   end
   s.add_dependency 'histogram'
-  s.required_ruby_version = '>= 2.4.0'
+  s.required_ruby_version = '>= 2.5.0'
 
   s.add_development_dependency 'rake'
   s.add_development_dependency 'minitest-reporters'
   s.add_development_dependency 'simplecov'
-  s.add_development_dependency 'yard', '~> 0.9.25'
+  s.add_development_dependency 'yard', '~> 0.9.28'
+
+  s.metadata['rubygems_mfa_required'] = 'true'
 end

data/lib/gruff/accumulator_bar.rb

@@ -19,7 +19,7 @@ private
   def setup_data
     raise(Gruff::IncorrectNumberOfDatasetsException) unless store.length == 1
 
-    accum_array = store.data.first.points[0..-2].reduce([0]) { |a, v| a << a.last + v }
+    accum_array = store.data.first.points[0..-2].reduce([0]) { |a, v| a << (a.last + v) }
     data 'Accumulator', accum_array
     set_colors
     store.reverse!

data/lib/gruff/area.rb

@@ -30,15 +30,17 @@ private
   end
 
   def draw_graph
-    x_increment = @graph_width / (column_count - 1).to_f
+    x_increment = @graph_width / (column_count - 1)
 
     store.norm_data.each do |data_row|
+      next if data_row.points.empty?
+
       poly_points = []
 
       data_row.points.each_with_index do |data_point, index|
         # Use incremented x and scaled y
         new_x = @graph_left + (x_increment * index)
-        new_y = @graph_top + (@graph_height - data_point * @graph_height)
+        new_y = @graph_top + (@graph_height - (data_point * @graph_height))
 
         poly_points << new_x
         poly_points << new_y
@@ -48,9 +50,9 @@ private
 
       # Add closing points, draw polygon
       poly_points << @graph_right
-      poly_points << @graph_bottom - 1
+      poly_points << (@graph_bottom - 1)
       poly_points << @graph_left
-      poly_points << @graph_bottom - 1
+      poly_points << (@graph_bottom - 1)
 
       Gruff::Renderer::Polygon.new(renderer, color: data_row.color, width: @stroke_width, opacity: @fill_opacity).render(poly_points)
     end

data/lib/gruff/bar.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'helper/bar_mixin'
+
 #
 # Gruff::Bar provide a bar graph that presents categorical data
 # with rectangular bars.
@@ -16,6 +18,8 @@
 #   g.write('bar.png')
 #
 class Gruff::Bar < Gruff::Base
+  include BarMixin
+
   # Spacing factor applied between bars.
   attr_writer :bar_spacing
 
@@ -33,6 +37,10 @@ class Gruff::Bar < Gruff::Base
   # Prevent drawing of column labels below a bar graph.  Default is +false+.
   attr_writer :hide_labels
 
+  # Value to avoid completely overwriting the coordinate axis
+  AXIS_MARGIN = 0.5
+  private_constant :AXIS_MARGIN
+
   # Can be used to adjust the spaces between the bars.
   # Accepts values between 0.00 and 1.00 where 0.00 means no spacing at all
   # and 1 means that each bars' width is nearly 0 (so each bar is a simple
@@ -70,15 +78,26 @@ private
   end
 
   def hide_left_label_area?
-    @hide_line_markers
+    @hide_line_markers && @y_axis_label.nil?
   end
 
   def hide_bottom_label_area?
-    hide_labels?
+    hide_labels? && @x_axis_label.nil? && @legend_at_bottom == false
   end
 
-  # Value to avoid completely overwriting the coordinate axis
-  AXIS_MARGIN = 0.5
+  def setup_graph_measurements
+    super
+    return if @hide_line_markers
+
+    if @show_labels_for_bar_values
+      if maximum_value >= 0
+        _, metrics = Gruff::BarValueLabel.metrics(maximum_value, @label_formatting, proc_text_metrics)
+        @graph_top += metrics.height
+      end
+
+      @graph_height = @graph_bottom - @graph_top
+    end
+  end
 
   def draw_graph
     # Setup spacing.
@@ -86,8 +105,8 @@ private
     # Columns sit side-by-side.
     @bar_spacing ||= @spacing_factor # space between the bars
 
-    bar_width = (@graph_width - calculate_spacing) / (column_count * store.length).to_f
-    padding = (bar_width * (1 - @bar_spacing)) / 2
+    bar_width = (@graph_width - calculate_spacing) / (column_count * store.length)
+    padding = (bar_width * (1 - @bar_spacing)) / 2.0
 
     # Setup the BarConversion Object
     conversion = Gruff::BarConversion.new(
@@ -95,34 +114,33 @@ private
       minimum_value: minimum_value, maximum_value: maximum_value, spread: @spread
     )
 
-    # iterate over all normalised data
-    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
-
-        # Use incremented x and scaled y
-        # x
-        left_x = @graph_left + (bar_width * (row_index + point_index + ((store.length - 1) * point_index))) + padding + group_spacing
-        right_x = left_x + bar_width * @bar_spacing
-        # y
-        left_y, right_y = conversion.get_top_bottom_scaled(data_point)
-
-        # create new bar
-        rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: data_row.color)
-        rect_renderer.render(left_x, left_y - AXIS_MARGIN, right_x, right_y - AXIS_MARGIN)
-
-        # Calculate center based on bar_width and current row
-        label_center = @graph_left + group_spacing + (store.length * bar_width * point_index) + (store.length * bar_width / 2.0)
-
-        # 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::Bar.new([left_x, left_y, right_x, right_y], store.data[row_index].points[point_index])
-          bar_value_label.prepare_rendering(@label_formatting, bar_width) do |x, y, text|
-            draw_value_label(x, y, text)
+    group_spacing = @group_spacing * @scale
+    group_left_x = @graph_left
+
+    normalized_group_bars.each_with_index do |group_bars, group_index|
+      right_x = 0
+      group_bars.each_with_index do |bar, index|
+        left_x = group_left_x + (bar_width * index) + padding
+        right_x = left_x + (bar_width * @bar_spacing)
+
+        top_y, bottom_y = conversion.get_top_bottom_scaled(bar.point)
+        if bar.point != 0
+          rect_renderer = Gruff::Renderer::Rectangle.new(renderer, color: bar.color)
+          rect_renderer.render(left_x, bottom_y - AXIS_MARGIN, right_x, top_y)
+        end
+
+        if @show_labels_for_bar_values && bar.value
+          bar_value_label = Gruff::BarValueLabel::Bar.new([left_x, top_y, right_x, bottom_y], bar.value)
+          bar_value_label.prepare_rendering(@label_formatting, proc_text_metrics) do |x, y, text, _text_width, text_height|
+            draw_value_label(bar_width * @bar_spacing, text_height, x, y, text)
           end
         end
       end
+
+      label_center = group_left_x + ((right_x - group_left_x) / 2.0)
+      draw_label(label_center, group_index)
+
+      group_left_x = right_x + padding + group_spacing
     end
 
     # Draw the last label if requested
@@ -132,4 +150,8 @@ private
   def calculate_spacing
     @scale * @group_spacing * (column_count - 1)
   end
+
+  def proc_text_metrics
+    ->(text) { text_metrics(@marker_font, text) }
+  end
 end