benchmark_driver 0.14.3 → 0.14.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 789cf1e0c9ef5f42aa6d5c8239a68ee601f7e8fefb063f3feac921faa7eb4b15
-  data.tar.gz: 8896c6c8f1983643bc81d70979ed7bce3c872276787d1d6b9f409ae2c27a7b26
+  metadata.gz: 75d01094d334aa05e52228bf11f65ff3a3f13178fb367fba125bdb871769dc4d
+  data.tar.gz: 68fa671585d87f7a3b06d4e15900cc2b565b9a504e7a6bf6fd63ac310104109c
 SHA512:
-  metadata.gz: c3ea314cf74d1ffa3c02904e1de9e9753cdd7f38724b3714b0d1f35a87252eccb6d3b47d2bc12bf60752c6275b5c33ba922194ab63c695637476398dc7698b21
-  data.tar.gz: 1a0f749bad759c7d8b3c6397ad78ac0f17ba62ef6e7758b4d122b4230b226176c956d17bd4c630d8345cc09ec0b154b8585739fb18a42fdbe4838ef659fd48fe
+  metadata.gz: 7147d203e882dccb92446fb901839e5450119c4ee65d38c20e025c1f557cf558ace84e066566a316497c097676d4d2276426befdc87e1021bac6bb19cd2d48d3
+  data.tar.gz: c2a431b5c1fa601c0e679125a4c9e5040698f8fb69405270988fbdc1057c6ccb1b350e4164474e5c8197970fff1f6688fff10bd9f3a626b4162747659e7e609d

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# v0.14.4
+
+- Fix runners and outputs to distinguish 0.0 and ERROR value
+
 # v0.14.3
 
 - Support loading runner plugin

data/lib/benchmark_driver/metric.rb

@@ -35,7 +35,7 @@ module BenchmarkDriver
     :environment, # @param [Hash] - Any other key -> value pairs to express the benchmark context
     defaults: { environment: {} },
   )
-  Result::ERROR = 0.0
+  Result::ERROR = 0
 
   # A kind of thing to be measured
   Metric = ::BenchmarkDriver::Struct.new(

data/lib/benchmark_driver/output/compare.rb

@@ -116,10 +116,12 @@ class BenchmarkDriver::Output::Compare
   end
 
   def humanize(value, width = 10)
-    if value == BenchmarkDriver::Result::ERROR
+    if BenchmarkDriver::Result::ERROR.equal?(value)
       return " %#{width}s" % 'ERROR'
+    elsif value == 0.0
+      return " %#{width}.3f" % 0.0
     elsif value < 0
-      raise ArgumentError.new("Non positive value: #{value.inspect}")
+      raise ArgumentError.new("Negative value: #{value.inspect}")
     end
 
     scale = (Math.log10(value) / 3).to_i

data/lib/benchmark_driver/output/markdown.rb

@@ -82,7 +82,11 @@ class BenchmarkDriver::Output::Markdown
   end
 
   def humanize(value)
-    if value < 0
+    if BenchmarkDriver::Result::ERROR.equal?(value)
+      return " %#{NAME_LENGTH}s" % 'ERROR'
+    elsif value == 0.0
+      return " %#{NAME_LENGTH}.3f" % 0.0
+    elsif value < 0
       raise ArgumentError.new("Negative value: #{value.inspect}")
     end
 

data/lib/benchmark_driver/output/simple.rb

@@ -77,8 +77,10 @@ class BenchmarkDriver::Output::Simple
   end
 
   def humanize(value)
-    if value == BenchmarkDriver::Result::ERROR
-      return 'ERROR'
+    if BenchmarkDriver::Result::ERROR.equal?(value)
+      return " %#{NAME_LENGTH}s" % 'ERROR'
+    elsif value == 0.0
+      return " %#{NAME_LENGTH}.3f" % 0.0
     elsif value < 0
       raise ArgumentError.new("Negative value: #{value.inspect}")
     end

data/lib/benchmark_driver/runner/ips.rb

@@ -123,7 +123,7 @@ class BenchmarkDriver::Runner::Ips
 
   # Overridden by BenchmarkDriver::Runner::Time
   def value_duration(duration:, loop_count:)
-    if duration == BenchmarkDriver::Result::ERROR
+    if BenchmarkDriver::Result::ERROR.equal?(duration)
       [BenchmarkDriver::Result::ERROR, BenchmarkDriver::Result::ERROR]
     else
       [loop_count.to_f / duration, duration]

data/lib/benchmark_driver/runner/once.rb

@@ -35,7 +35,7 @@ class BenchmarkDriver::Runner::Once
         @output.with_job(name: job.name) do
           job.runnable_contexts(@contexts).each do |context|
             duration = run_benchmark(job, context: context) # no repeat support
-            if duration == BenchmarkDriver::Result::ERROR
+            if BenchmarkDriver::Result::ERROR.equal?(duration)
               value = BenchmarkDriver::Result::ERROR
             else
               value = 1.0 / duration

data/lib/benchmark_driver/version.rb

@@ -1,3 +1,3 @@
 module BenchmarkDriver
-  VERSION = '0.14.3'
+  VERSION = '0.14.4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: benchmark_driver
 version: !ruby/object:Gem::Version
-  version: 0.14.3
+  version: 0.14.4
 platform: ruby
 authors:
 - Takashi Kokubun