Skip to content

Commit

Permalink
Improve plots for :alloc
Browse files Browse the repository at this point in the history
  • Loading branch information
Azzaare committed Jun 10, 2024
1 parent cdcb050 commit e75af15
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 9 deletions.
10 changes: 3 additions & 7 deletions ext/MakieExt/allocs.jl
Original file line number Diff line number Diff line change
Expand Up @@ -73,16 +73,12 @@ function PerfChecker.checkres_to_scatterlines(
step = 0.02 * (lx - 1)
diff = (1 - ly) * step / 2.0

@info "Plotting allocations" lx ly step diff

for (keys, values) in di
xs = [values[i][1] for i in eachindex(values)] .+ diff
ys = [versions[values[i][2]] for i in eachindex(values)]
scatterlines!(f[1, 1], ys, xs, label = keys, color = (colors[i], 0.6))
ys = [values[i][1] for i in eachindex(values)]
xs = [versions[values[i][2]] for i in eachindex(values)] .+ diff
scatterlines!(f[1, 1], xs, ys, label = keys, color = (colors[i], 1.0))
i += 1
diff += step

@info "Plotting allocations" xs ys
end
ax.title = x.pkgs[1].name
Legend(f[1, 2], ax)
Expand Down
1 change: 0 additions & 1 deletion ext/MakieExt/bench.jl
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ function PerfChecker.checkres_to_scatterlines(
ax.xlabel = "versions"
ax.ylabel = "ratio"
ax.title = "Evolution for $(x.pkgs[1].name) (via BenchmarkTools.jl)"
# ylims!(; high = max)
f[1, 2] = Legend(f, ax)
return f
end
Expand Down
1 change: 0 additions & 1 deletion ext/MakieExt/chair.jl
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ function PerfChecker.checkres_to_scatterlines(
ax.xlabel = "versions"
ax.ylabel = "ratio"
ax.title = "Evolution for $(x.pkgs[1].name) (via Chairmarks.jl)"
# ylims!(; high = max)
f[1, 2] = Legend(f, ax)
return f
end
Expand Down
Binary file modified perf/PatternFolds/visuals/allocs_evolution.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit e75af15

Please sign in to comment.