benchmark_driver 0.15.9 → 0.15.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e99ed161f437a1c8e08847db6bee7f8ada9773fc527d1be4aaa6436c18ce16d3
-  data.tar.gz: cc6aa7374390d0451f8d75451676481b032d1d7d8fe4bbdc442e77b4c2fad19a
+  metadata.gz: 6d1052380e38062b6a13370022a91959621902cf2d08263d29fc504c9297655f
+  data.tar.gz: 32067954ee2170d157f974ead3dce80aa7a2d0dca6f322f2aae053693775a656
 SHA512:
-  metadata.gz: 2a89c19b24c61a01b97a6814c6a47185fb19ce81ea8afedbfefa812610aa4774179a128d8e365435c58f967d84faf61ce7628b8672db65e7962a5ee950045db6
-  data.tar.gz: 71417fee7eadda1c26f6d0cab1c9c7563eb1d8da011f4b4b2c2e1701eaba573bdbf67230fb74034cc08353396bc20e6573ea79cc8acb8ab17f7c6cd01285cb21
+  metadata.gz: d8d5417b30e026ea54fe080f434b89fde4c36f8136b3ea082b972d032730ed421d019c5eb2213efbcf5d0e905cc41ced746db31a0af5f1f3c8edfb1ecfb16320
+  data.tar.gz: 72da43bfd0d31749296ceb806da6210a86d55ad1046a06a8b2664e023c8ace1f782e1df1f7919205d4e9e95abc736440360fab1a1f8c6824028abc0698ebd2a8

data/CHANGELOG.md

@@ -1,3 +1,23 @@
+# v0.15.14
+
+- YAML's `type` key allows a value with `/` to specify a relative path of a runner plugin
+
+# v0.15.13
+
+- Show a command and stdout on `-vv` for `ips`, `time`, and `block` runner
+
+# v0.15.12
+
+- Show comparison on `--output=markdown` when `--output-compare` is also specified
+
+# v0.15.11
+
+- Correctly calculate an average result on `--repeat-result=average`
+
+# v0.15.10
+
+- Right-justify calculated results in `markdown` output
+
 # v0.15.9
 
 - Prefer an exact match in RVM version selection

data/exe/benchmark-driver

@@ -124,7 +124,7 @@ jobs = config.paths.flat_map do |path|
 
   begin
     # `working_directory` is YAML-specific special parameter, mainly for "command_stdout"
-    BenchmarkDriver::JobParser.parse(job, default_params: { working_directory: File.dirname(path) })
+    BenchmarkDriver::JobParser.parse(job, working_directory: File.dirname(path))
   rescue ArgumentError
     $stderr.puts "benchmark-driver: Failed to parse #{path.dump}."
     $stderr.puts '  YAML format may be wrong. See error below:'

data/lib/benchmark_driver/job_parser.rb

@@ -3,24 +3,27 @@ require 'benchmark_driver/runner'
 module BenchmarkDriver
   class << JobParser = Module.new
     # @param [Hash] config
-    # @param [Hash] default_params - Special default values not written in job definition
-    def parse(config, default_params: {})
+    # @param [Hash] working_directory - YAML-specific special parameter for "command_stdout" and a relative path in type
+    def parse(config, working_directory: nil)
       config = symbolize_keys(config)
       type = config.fetch(:type)
       if !type.is_a?(String)
         raise ArgumentError.new("Invalid type: #{config[:type].inspect} (expected String)")
-      elsif !type.match(/\A[A-Za-z0-9_]+\z/)
-        raise ArgumentError.new("Invalid type: #{config[:type].inspect} (expected to include only [A-Za-z0-9_])")
+      elsif !type.match(/\A[A-Za-z0-9_\/]+\z/)
+        raise ArgumentError.new("Invalid type: #{config[:type].inspect} (expected to include only [A-Za-z0-9_\/])")
       end
       config.delete(:type)
 
       # Dynamic dispatch for plugin support
-      require "benchmark_driver/runner/#{type}"
+      if type.include?('/')
+        require File.join(working_directory || '.', type)
+        type = File.basename(type)
+      else
+        require "benchmark_driver/runner/#{type}"
+      end
       job = ::BenchmarkDriver.const_get("Runner::#{camelize(type)}::JobParser", false).parse(**config)
-      default_params.each do |key, value|
-        if job.respond_to?(key) && job.respond_to?("#{key}=") && job.public_send(key).nil?
-          job.public_send("#{key}=", value)
-        end
+      if job.respond_to?(:working_directory) && job.respond_to?(:working_directory=) && job.working_directory.nil?
+        job.working_directory = working_directory
       end
       job
     end

data/lib/benchmark_driver/output/compare.rb

@@ -117,17 +117,17 @@ class BenchmarkDriver::Output::Compare
 
   def humanize(value, width = 10)
     if BenchmarkDriver::Result::ERROR.equal?(value)
-      return " %#{width}s" % 'ERROR'
+      return sprintf(" %*s", width, 'ERROR')
     elsif value == 0.0
-      return " %#{width}.3f" % 0.0
+      return sprintf(" %*.3f", width, 0.0)
     elsif value < 0
       raise ArgumentError.new("Negative value: #{value.inspect}")
     end
 
     scale = (Math.log10(value) / 3).to_i
-    return "%#{width}s" % value.to_s if scale < 0 # like 1.23e-04
+    return sprintf("%*s", width, value.to_s) if scale < 0 # like 1.23e-04
 
-    prefix = "%#{width}.3f" % (value.to_f / (1000 ** scale))
+    prefix = sprintf("%*.3f", width, (value.to_f / (1000 ** scale)))
     suffix =
       case scale
       when 1; 'k'
@@ -173,7 +173,7 @@ class BenchmarkDriver::Output::Compare
     $stdout.puts "\nComparison:"
 
     @job_context_result.each do |job, context_result|
-      $stdout.puts("%#{@name_length + 2 + 11}s" % job)
+      $stdout.printf("%*s\n", @name_length + 2 + 11, job)
       results = context_result.flat_map do |context, result|
         result.values.values.map { |value| Result.new(job: job, value: value, context: context) }
       end
@@ -188,7 +188,7 @@ class BenchmarkDriver::Output::Compare
 
     unless BenchmarkDriver::Result::ERROR.equal?(bottom)
       ratio = top / bottom
-      "- %.2fx  #{@metrics.first.worse_word}" % ratio
+      sprintf("- %.2fx  %s", ratio, @metrics.first.worse_word)
     end
   end
 
@@ -211,7 +211,7 @@ class BenchmarkDriver::Output::Compare
       else
         name = result.job
       end
-      $stdout.puts("%#{@name_length}s: %11.1f %s #{slower}" % [name, result.value, @metrics.first.unit])
+      $stdout.printf("%*s: %11.1f %s %s\n", @name_length, name, result.value, @metrics.first.unit, slower)
     end
     $stdout.puts
   end

data/lib/benchmark_driver/output/markdown.rb

@@ -1,13 +1,19 @@
 class BenchmarkDriver::Output::Markdown
   NAME_LENGTH = 8
 
+  OPTIONS = {
+    compare: ['--output-compare', 'Show comparison between results'],
+  }
+
   # @param [Array<BenchmarkDriver::Metric>] metrics
   # @param [Array<BenchmarkDriver::Job>] jobs
   # @param [Array<BenchmarkDriver::Context>] contexts
-  def initialize(metrics:, jobs:, contexts:)
+  def initialize(metrics:, jobs:, contexts:, options:)
     @metrics = metrics
+    @contexts = contexts
     @context_names = contexts.map(&:name)
     @name_length = jobs.map(&:name).map(&:size).max
+    @compare = options.fetch(:compare, false)
   end
 
   def with_warmup(&block)
@@ -28,14 +34,15 @@ class BenchmarkDriver::Output::Markdown
       # Show executable names
       $stdout.print("|#{' ' * @name_length}  ")
       @context_names.each do |context_name|
-        $stdout.print("|%#{NAME_LENGTH}s" % context_name) # same size as humanize
+        $stdout.printf("|%*s", NAME_LENGTH, context_name) # same size as humanize
       end
       $stdout.puts('|')
 
       # Show header separator
       $stdout.print("|:#{'-' * (@name_length - 1)}--")
       @context_names.each do |context_name|
-        $stdout.print("|:#{'-' * (NAME_LENGTH - 1)}") # same size as humanize
+        length = [context_name.length, NAME_LENGTH].max
+        $stdout.print("|#{'-' * (length - 1)}:") # same size as humanize
       end
       $stdout.puts('|')
 
@@ -48,24 +55,33 @@ class BenchmarkDriver::Output::Markdown
   # @param [BenchmarkDriver::Job] job
   def with_job(job, &block)
     if @with_benchmark
-      $stdout.print("|%-#{@name_length}s  " % job.name)
+      @job_context_result = {} if @context_names.size > 1
+
+      $stdout.printf("|%-*s  ", @name_length, job.name)
     end
     block.call
   ensure
     if @with_benchmark
       $stdout.puts('|')
+      compare_executables if @compare && @context_names.size > 1
     end
   end
 
   # @param [BenchmarkDriver::Context] context
   def with_context(context, &block)
+    @context = context
     block.call
   end
 
   # @param [BenchmarkDriver::Result] result
   def report(result)
+    if defined?(@job_context_result)
+      @job_context_result[@context] = result
+    end
+
     if @with_benchmark
-      $stdout.print("|%#{NAME_LENGTH}s" % humanize(result.values.fetch(@metrics.first)))
+      length = [NAME_LENGTH, @context.name.length].max
+      $stdout.printf("|%*s", length, humanize(result.values.fetch(@metrics.first)))
     else
       $stdout.print '.'
     end
@@ -83,15 +99,15 @@ class BenchmarkDriver::Output::Markdown
 
   def humanize(value)
     if BenchmarkDriver::Result::ERROR.equal?(value)
-      return "%#{NAME_LENGTH}s" % 'ERROR'
+      return sprintf("%*s", NAME_LENGTH, 'ERROR')
     elsif value == 0.0
-      return "%#{NAME_LENGTH}.3f" % 0.0
+      return sprintf("%*.3f", NAME_LENGTH, 0.0)
     elsif value < 0
       raise ArgumentError.new("Negative value: #{value.inspect}")
     end
 
     scale = (Math.log10(value) / 3).to_i
-    prefix = "%#{NAME_LENGTH - 1}.3f" % (value.to_f / (1000 ** scale))
+    prefix = sprintf("%*.3f", NAME_LENGTH - 1, (value.to_f / (1000 ** scale)))
     suffix =
       case scale
       when 1; 'k'
@@ -104,4 +120,30 @@ class BenchmarkDriver::Output::Markdown
       end
     "#{prefix}#{suffix}"
   end
+
+  def compare_executables
+    order = @metrics.first.larger_better ? :min_by : :max_by
+    worst, worst_result = @job_context_result.__send__(order) do |_, result|
+      result.values.first[1]
+    end
+    worst_result = worst_result.values.first[1]
+    $stdout.print("|", " " * (@name_length + 2))
+    @job_context_result.each do |context, result|
+      if context == worst
+        result = '-'
+      else
+        result = result.values.first[1]
+        if order == :min_by
+          result = result.fdiv(worst_result)
+        else
+          result = best_result.fdiv(worst_result)
+        end
+        result = sprintf("%.2fx", result)
+      end
+      length = [context.name.length, NAME_LENGTH].max
+      $stdout.printf("|%*s", length, result)
+    end
+    $stdout.puts('|')
+  end
+
 end

data/lib/benchmark_driver/repeater.rb

@@ -54,7 +54,7 @@ module BenchmarkDriver
           [values.map { |v| v[0] }.inject(&:+) / values.size.to_f, *rest]
         when :average
           values.first.size.times.map do |index|
-            values.map { |v| v[index] }.inject(&:+) / values.first.size.to_f
+            values.map { |v| v[index] }.inject(&:+) / values.size.to_f
           end
         else
           raise "unexpected rest_on_average #{rest_on_average.inspect}"

data/lib/benchmark_driver/runner/ips.rb

@@ -107,8 +107,8 @@ class BenchmarkDriver::Runner::Ips
 
     duration = Tempfile.open(['benchmark_driver-', '.rb']) do |f|
       with_script(benchmark.render(result: f.path)) do |path|
-        IO.popen([*context.executable.command, path], &:read) # TODO: print stdout if verbose=2
-        if $?.success? && ((value = Float(f.read)) > 0)
+        success = execute(*context.executable.command, path, exception: false)
+        if success && ((value = Float(f.read)) > 0)
           value
         else
           BenchmarkDriver::Result::ERROR
@@ -137,10 +137,7 @@ class BenchmarkDriver::Runner::Ips
   end
 
   def with_script(script)
-    if @config.verbose >= 2
-      sep = '-' * 30
-      $stdout.puts "\n\n#{sep}[Script begin]#{sep}\n#{script}#{sep}[Script end]#{sep}\n\n"
-    end
+    debug_output('Script', script) if @config.verbose >= 2
 
     Tempfile.open(['benchmark_driver-', '.rb']) do |f|
       f.puts script
@@ -149,11 +146,20 @@ class BenchmarkDriver::Runner::Ips
     end
   end
 
-  def execute(*args)
-    IO.popen(args, &:read) # TODO: print stdout if verbose=2
-    unless $?.success?
-      raise "Failed to execute: #{args.shelljoin} (status: #{$?.exitstatus})"
+  def execute(*args, exception: true)
+    $stderr.puts "$ #{args.shelljoin}" if @config.verbose >= 2
+
+    stdout = IO.popen(args, &:read)
+    debug_output('Command output', stdout) if @config.verbose >= 2
+    if exception && !$?.success?
+      raise "Failed to execute: #{args.shelljoin} (status: #{$?})"
     end
+    $?.success?
+  end
+
+  def debug_output(name, text)
+    sep = '-' * 30
+    $stdout.puts "\n\n#{sep}[#{name} begin]#{sep}\n#{text}#{sep}[#{name} end]#{sep}\n\n"
   end
 
   WarmupScript = ::BenchmarkDriver::Struct.new(:preludes, :script, :teardown, :loop_count, :first_warmup_duration, :second_warmup_duration) do

data/lib/benchmark_driver/version.rb

@@ -1,3 +1,3 @@
 module BenchmarkDriver
-  VERSION = '0.15.9'
+  VERSION = '0.15.14'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: benchmark_driver
 version: !ruby/object:Gem::Version
-  version: 0.15.9
+  version: 0.15.14
 platform: ruby
 authors:
 - Takashi Kokubun
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-01-04 00:00:00.000000000 Z
+date: 2020-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -146,7 +146,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.0.pre1
 signing_key: 
 specification_version: 4
 summary: Fully-featured accurate benchmark driver for Ruby