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

Dodge stack for ggplot #6328

Closed
wants to merge 10 commits into from
Closed

Dodge stack for ggplot #6328

wants to merge 10 commits into from

Conversation

szkabel
Copy link

@szkabel szkabel commented Feb 11, 2025

I have been using this extension of mine for a while for dodging and stacking together. Even though it is a functionality only useful for column geoms, it still seems to be a topic of interest based on a recently opened feature request. I myself implemented it because of a need in a research project.

The idea of the functionality is that after dodging the rest of the values are stacked. For us this was really beneficial. Also the addition is written so that it doesn't change any previous behavior.

I have made a description on why I think this is useful and why previously this may have caused confusion here: https://rpubs.com/szkabel/dodgeStackDemo

@teunbrand
Copy link
Collaborator

Hi there, thanks for putting together this PR. Unfortunately, we don't think this functionality is a good fit for ggplot2 and would instead recommend implementing this functionality in an extension package. We nonetheless appreciate the motivation and time invested in preparing this PR and hope this doesn't impede any future contributions!

@teunbrand teunbrand closed this Feb 12, 2025
@clauswilke
Copy link
Member

@szkabel To follow up on Teun's point: I also think that this should not be part of the standard position_dodge() position adjustment but instead go into its own. And once you do that you don't have to add the code to ggplot2 main. Something like ggforce might be more appropriate, if you don't want to write your own extension package for just one single function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants