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

doesn't supporting the github version of ggplot2 #33

Open
xiangpin opened this issue Aug 24, 2022 · 2 comments
Open

doesn't supporting the github version of ggplot2 #33

xiangpin opened this issue Aug 24, 2022 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@xiangpin
Copy link

Hello
Thanks for the great package. the ggplot2 is a very popular package. ggupset seem to be developed based on it. But now ggplot2
has some significant changes. and the ggupset is compatible with it.

> library(ggplot2)
> library(ggupset)
> packageVersion('ggplot2')
[1] ‘3.3.6.9000’
> packageVersion('ggupset')
[1] ‘0.3.0’
> example(scale_x_upset)

scl_x_> library(ggplot2)

scl_x_> ggplot(tidy_movies[1:100, ], aes(x=Genres)) +
scl_x_    geom_bar() +
scl_x_    scale_x_upset(reverse = TRUE, sets=c("Drama", "Action"))
Hit <Return> to see next plot:
Error: Discrete value supplied to continuous scale
> example(theme_combmatrix)

thm_cm> library(ggplot2)

thm_cm> # Ensure that the y-axis label is next to the axis by setting
thm_cm> # combmatrix.label.make_space to FALSE
thm_cm> ggplot(tidy_movies[1:100, ], aes(x=Genres)) +
thm_cm    geom_bar() +
thm_cm    scale_x_upset() +
thm_cm    theme_combmatrix(combmatrix.label.text = element_text(color = "black", size=15),
thm_cm                     combmatrix.label.make_space = FALSE,
thm_cm                     plot.margin = unit(c(1.5, 1.5, 1.5, 65), "pt"))
Hit <Return> to see next plot:
Error: Discrete value supplied to continuous scale
@const-ae
Copy link
Owner

Hi Shuangbin Xu,

thanks for alerting me to this. I agree it's worrisome if the next ggplot release will break ggupset and I will keep an eye on it. Unfortunately, I currently don't have the spare capacity to investigate this issue myself and figure out if it is a simple 1-line fix or will entail a more complete rewrite of ggupset.
Sometimes the ggplot developers provide some advice how to transition to the new version if they include breaking changes. So I will take a wait-and-see approach for now.

Best,
Constantin

@const-ae const-ae added the help wanted Extra attention is needed label Aug 24, 2022
@xiangpin
Copy link
Author

ok, thanks, I found this issue seems similar to the ggside.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants