Fixes for visualisation and some perf output

JuliaConstraints · Jun 5, 2024 · e8caab8 · e8caab8
1 parent 0ae3434
commit e8caab8
Show file tree

Hide file tree

Showing 27 changed files with 11 additions and 8 deletions.
diff --git a/ext/MakieExt/allocs.jl b/ext/MakieExt/allocs.jl
@@ -52,7 +52,7 @@ function PerfChecker.checkres_to_scatterlines(
     versionnums = [x.pkgs[i].version for i in eachindex(x.pkgs)]
     f = Figure()
     ax = Axis(f[1, 1])
-    ax.yscale = log2
+    ax.yscale = Makie.pseudolog10
     ax.xticks = (eachindex(versionnums), string.(versionnums))
     ax.xlabel = "versions"
     ax.ylabel = "bytes"

diff --git a/ext/MakieExt/bench.jl b/ext/MakieExt/bench.jl
@@ -30,7 +30,7 @@ function PerfChecker.checkres_to_scatterlines(
         end
         scatterlines!(xs, ys, label = string(props[i]), color = (colors[i], 0.4))
     end
-    ax.yscale = log2
+    ax.yscale = Makie.pseudolog10
     ax.xticks = (eachindex(versionnums), string.(versionnums))
     ax.xlabel = "versions"
     ax.ylabel = "ratio"

diff --git a/ext/MakieExt/chair.jl b/ext/MakieExt/chair.jl
@@ -30,7 +30,7 @@ function PerfChecker.checkres_to_scatterlines(
         end
         scatterlines!(xs, ys, label = string(props[i]), color = (colors[i], 0.4))
     end
-    ax.yscale = log2
+    ax.yscale = Makie.pseudolog10
     ax.xticks = (eachindex(versionnums), string.(versionnums))
     ax.xlabel = "versions"
     ax.ylabel = "ratio"

diff --git a/ext/MakieExt/plotutils.jl b/ext/MakieExt/plotutils.jl
@@ -24,4 +24,4 @@ function smart_paths(paths)
 end
 
 # Fix the maximum limit in plot by some epsilon for log scale plots
-ϵ(x; adjust = 0.1 * log(2)) = x * exp(adjust)
+ϵ(x; adjust = 0.1 * log(10)) = x * exp(adjust)
diff --git a/perf/GLM/visuals/bench_boxplots_allocs.png b/perf/GLM/visuals/bench_boxplots_allocs.png
diff --git a/perf/GLM/visuals/bench_boxplots_gctimes.png b/perf/GLM/visuals/bench_boxplots_gctimes.png
diff --git a/perf/GLM/visuals/bench_boxplots_memory.png b/perf/GLM/visuals/bench_boxplots_memory.png
diff --git a/perf/GLM/visuals/bench_boxplots_times.png b/perf/GLM/visuals/bench_boxplots_times.png
diff --git a/perf/GLM/visuals/bench_evolution.png b/perf/GLM/visuals/bench_evolution.png
diff --git a/perf/GLM/visuals/chair_boxplots_allocs.png b/perf/GLM/visuals/chair_boxplots_allocs.png
diff --git a/perf/GLM/visuals/chair_boxplots_bytes.png b/perf/GLM/visuals/chair_boxplots_bytes.png
diff --git a/perf/GLM/visuals/chair_boxplots_gctimes.png b/perf/GLM/visuals/chair_boxplots_gctimes.png
diff --git a/perf/GLM/visuals/chair_boxplots_times.png b/perf/GLM/visuals/chair_boxplots_times.png
diff --git a/perf/GLM/visuals/chair_evolution.png b/perf/GLM/visuals/chair_evolution.png
diff --git a/perf/PatternFolds/visuals/allocs_evolution.png b/perf/PatternFolds/visuals/allocs_evolution.png
diff --git a/perf/PatternFolds/visuals/bench_boxplots_allocs.png b/perf/PatternFolds/visuals/bench_boxplots_allocs.png
diff --git a/perf/PatternFolds/visuals/bench_boxplots_gctimes.png b/perf/PatternFolds/visuals/bench_boxplots_gctimes.png
diff --git a/perf/PatternFolds/visuals/bench_boxplots_memory.png b/perf/PatternFolds/visuals/bench_boxplots_memory.png
diff --git a/perf/PatternFolds/visuals/bench_boxplots_times.png b/perf/PatternFolds/visuals/bench_boxplots_times.png
diff --git a/perf/PatternFolds/visuals/bench_evolution.png b/perf/PatternFolds/visuals/bench_evolution.png
diff --git a/perf/PatternFolds/visuals/chair_boxplots_allocs.png b/perf/PatternFolds/visuals/chair_boxplots_allocs.png
diff --git a/perf/PatternFolds/visuals/chair_boxplots_bytes.png b/perf/PatternFolds/visuals/chair_boxplots_bytes.png
diff --git a/perf/PatternFolds/visuals/chair_boxplots_gctimes.png b/perf/PatternFolds/visuals/chair_boxplots_gctimes.png
diff --git a/perf/PatternFolds/visuals/chair_boxplots_times.png b/perf/PatternFolds/visuals/chair_boxplots_times.png
diff --git a/perf/PatternFolds/visuals/chair_evolution.png b/perf/PatternFolds/visuals/chair_evolution.png
diff --git a/src/check.jl b/src/check.jl
@@ -60,15 +60,18 @@ function check_function(x::Symbol, d::Dict, block1, block2)
 
     for i in 1:len
         is_loaded = false
-        di[:current_spec] = pkgs[i]
-        di[:current_version] = pkgs[i].version
         if i ≤ length(pkgs)
+            di[:current_spec] = pkgs[i]
+            di[:current_version] = pkgs[i].version
             path = joinpath(di[:path], "metadata", "metadata.csv")
             fp = flatten_parameters(x, pkgs[i].name, pkgs[i].version, d[:tags])
             u = get_uuid() |> Base.UUID
             if in_metadata(path, fp, u)
                 is_loaded = true
             end
+        else
+            di[:current_spec] = di[:devops]
+            di[:current_version] = "dev"
         end
 
         if !is_loaded

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -11,6 +11,6 @@ using Test
         include("pattern_folds.jl")
     end
 
-    rm("test/metadata"; recursive = true, force = true)
-    rm("test/output"; recursive = true, force = true)
+    rm("metadata"; recursive = true, force = true)
+    rm("output"; recursive = true, force = true)
 end