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.

Sign up for GitHub

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

Jump to bottom

geom_violin() for bounded data #5493

Closed
eliocamp opened this issue Oct 25, 2023 · 1 comment · Fixed by #5494
Closed

geom_violin() for bounded data #5493

eliocamp opened this issue Oct 25, 2023 · 1 comment · Fixed by #5494

Comments

@eliocamp
Copy link
Contributor

It would be great if geom_violin() inherited geom_density()'s bounds argument to deal with bounded data. (See #3387)
@teunbrand
Copy link
Collaborator

Since both geom_density() and geom_violin() internally use ggplot2:::compute_density(), it should be just a matter of passing around an extra bounds argument in StatYdensity.

eliocamp added a commit to eliocamp/ggplot2 that referenced this issue Oct 25, 2023
@eliocamp
Adds bounds to geom_violin()
8e84477 
Closes tidyverse#5493
@eliocamp eliocamp mentioned this issue Oct 25, 2023
Merged
teunbrand pushed a commit that referenced this issue Oct 26, 2023
@eliocamp
Adds bounds to geom_violin() (#5494)
5a61e2e 
Closes #5493
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds bounds to geom_violin() eliocamp/ggplot2
2 participants
@eliocamp @teunbrand