Skip to content

Commit 4b6e9e3

Browse files
committed
Extract method to calculate the seepdups
1 parent 8013bba commit 4b6e9e3

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

lib/graph_renderer.rb

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,7 @@ def self.render(json_path, png_path, title_font_size: 16.0, legend_font_size: 12
3030
configure_graph(graph, ruby_descriptions, bench_names, title_font_size, legend_font_size, marker_font_size)
3131

3232
ruby_descriptions.each do |ruby, description|
33-
speedups = bench_names.map { |bench|
34-
baseline_times = data.fetch(baseline).fetch(bench).fetch("bench")
35-
times = data.fetch(ruby).fetch(bench).fetch("bench")
36-
Stats.new(baseline_times).mean / Stats.new(times).mean
37-
}
33+
speedups = calculate_speedups(data, baseline, ruby, bench_names)
3834
graph.data "#{ruby}: #{description}", speedups
3935
end
4036
graph.write(png_path)
@@ -62,4 +58,12 @@ def self.configure_graph(graph, ruby_descriptions, bench_names, title_font_size,
6258
graph.legend_font_size = legend_font_size
6359
graph.marker_font_size = marker_font_size
6460
end
61+
62+
def self.calculate_speedups(data, baseline, ruby, bench_names)
63+
bench_names.map { |bench|
64+
baseline_times = data.fetch(baseline).fetch(bench).fetch("bench")
65+
times = data.fetch(ruby).fetch(bench).fetch("bench")
66+
Stats.new(baseline_times).mean / Stats.new(times).mean
67+
}
68+
end
6569
end

0 commit comments

Comments
 (0)