Skip to content

Conversation

mwaskom
Copy link
Owner

@mwaskom mwaskom commented May 19, 2022

Fixes #2742

With the example from that issue:

N = 30
np.random.seed(3)
data = np.random.normal(size=(N,))
df = pd.DataFrame({'Before': data,
                   'After': data + 1})
df_long = pd.melt(df, value_vars=['Before', 'After'])
ax = sns.violinplot(x='variable', y='value', data=df_long, hue='variable', split=True, inner='box', cut=1)

image

@codecov
Copy link

codecov bot commented May 19, 2022

Codecov Report

Merging #2814 (d4fceee) into master (34b8873) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2814   +/-   ##
=======================================
  Coverage   97.96%   97.96%           
=======================================
  Files          67       67           
  Lines       21756    21761    +5     
=======================================
+ Hits        21313    21318    +5     
  Misses        443      443           
Impacted Files Coverage Δ
seaborn/categorical.py 98.24% <100.00%> (ø)
seaborn/tests/test_categorical.py 99.76% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34b8873...d4fceee. Read the comment docs.

@mwaskom mwaskom merged commit a166af2 into master May 22, 2022
@mwaskom mwaskom deleted the fix/unpaired_split_violins branch May 22, 2022 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

seaborn violinplot does not work right for 'inner=points' and 'inner=box' options

1 participant