From 027a2a68dfb9be30074f2cc8c7a200a0dce649ec Mon Sep 17 00:00:00 2001 From: Daniel Schwabeneder Date: Tue, 30 Jun 2020 22:55:03 +0200 Subject: [PATCH] Revert "Merge pull request #353 from Moelf/groupbar_dodge_log" This reverts commit d8ef19afcea0534224165f565fc3764ae48d358c. --- src/bar.jl | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/src/bar.jl b/src/bar.jl index 37a9989..aa34c51 100644 --- a/src/bar.jl +++ b/src/bar.jl @@ -12,8 +12,7 @@ grouped_xy(y::AbstractArray) = 1:size(y,1), y nr, nc = size(y) isstack = pop!(plotattributes, :bar_position, :dodge) == :stack - isylog = pop!(plotattributes, :yscale, :identity) ∈ (:log10, :log) - the_ylims = pop!(plotattributes, :ylims, (-Inf, Inf)) + isylog = pop!(plotattributes, :yscale, :identity) == :log10 # extract xnums and set default bar width. # might need to set xticks as well @@ -48,14 +47,13 @@ grouped_xy(y::AbstractArray) = 1:size(y,1), y end # compute fillrange - y, fr = groupedbar_fillrange(y) - fillrange := if isylog - if isstack - fill_bottom = min(minimum(y) / 100, the_ylims[1]) - # to fill the blank of the bottom segment depeding on if ylims - replace!(fr, 0 => fill_bottom) + fillrange := if isstack + y, fr = groupedbar_fillrange(y) + if isylog + # to fill the blank of the bottom segment + replace!(fr, 0=>eps(Float64)) else - max(minimum(y) / 100, the_ylims[1]) + fr end else get(plotattributes, :fillrange, nothing)