Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Jitter do something by default #3066

Merged
merged 3 commits into from
Oct 9, 2022
Merged

Make Jitter do something by default #3066

merged 3 commits into from
Oct 9, 2022

Conversation

mwaskom
Copy link
Owner

@mwaskom mwaskom commented Oct 9, 2022

This adds a small amount of width jitter (i.e., jitter along the orientation axis proportional to the spacing between marks) by default:

(
    so.Plot(penguins, "species", "body_mass_g")
    .add(so.Dots(), so.Jitter())
)

The default jitter is disabled if any of the other jitter parameters (x or y) are specified.

Additionally, added API examples.

@codecov
Copy link

codecov bot commented Oct 9, 2022

Codecov Report

Merging #3066 (e159427) into master (aa56714) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3066   +/-   ##
=======================================
  Coverage   98.38%   98.38%           
=======================================
  Files          74       74           
  Lines       23635    23648   +13     
=======================================
+ Hits        23254    23267   +13     
  Misses        381      381           
Impacted Files Coverage Δ
seaborn/_core/moves.py 100.00% <100.00%> (ø)
seaborn/_core/plot.py 99.31% <100.00%> (-0.01%) ⬇️
seaborn/_core/typing.py 100.00% <100.00%> (ø)
tests/_core/test_moves.py 100.00% <100.00%> (ø)

@mwaskom mwaskom merged commit 54cab15 into master Oct 9, 2022
@mwaskom mwaskom deleted the move/jitter branch October 9, 2022 23:31
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.

1 participant