For raincloud plot, jitter_width is documented, but cannot be used

[goulf-3m]

When I run

Makie.rainclouds([string(x) for x in rand(1:3, 50)], randn(50), jitter_width=0.05)

The error occurs

ERROR: Invalid attribute Scatter for plot type MakieCore.Plot{Makie.rainclouds}.

[ericphanson]

Just to say I have the same issue, though the error message is different:

Invalid attribute jitter_width for plot type MakieCore.Plot{Makie.rainclouds, Tuple{Vector{String}, Vector{Float64}}}.



The available plot attributes for MakieCore.Plot{Makie.rainclouds, Tuple{Vector{String}, Vector{Float64}}} are:



boxplot_nudge   color      hist_bins      show_boxplot_outliers  whiskerwidth

boxplot_width   cycle      markersize     show_median                        

center_boxplot  dodge      n_dodge        side                               

cloud_width     dodge_gap  orientation    strokewidth                        

clouds          gap        plot_boxplots  violin_limits                      





Generic attributes are:



clip_planes  dim_conversions  model      transformation  yautolimits        

cycle        label            rasterize  xautolimits     zautolimits        

[asinghvi17]

Makie.jl/src/basic_recipes/raincloud.jl

Lines 248 to 255 in a1c513b

	side_scatter_nudge_default = plot_boxplots ? 0.2 : 0.075
	jitter_width_default = 0.05
	# Scatter Plot Settings
	side_scatter_nudge = to_value(get(plot, :side_nudge, side_scatter_nudge_default))
	side_scatter_nudge < 0 && ArgumentError("`side_nudge` should be positive. Change `side` to :left, :right if you wish.")
	jitter_width = abs(to_value(get(plot, :jitter_width, jitter_width_default)))
	jitter_width < 0 && ArgumentError("`jitter_width` should be positive.")

[ericphanson]

not sure what what you mean @asinghvi17? I think the issue is the attribute system doesn't recognize it so it throws an error

[asinghvi17]

Ah I sent this before adding context 😅

Yeah, it's just that jitter_width and side_nudge are not listed as attributes. I think that's because they depend on some pre-calculated thing, but side_nudge could just be Makie.automatic by default and jitter_width can stay as is I think.

[ericphanson]

thanks @asinghvi17 !