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

change default_aes(size=...) to default_aes(linewidth=...) #28

Open
ammar-gla opened this issue Apr 26, 2023 · 0 comments
Open

change default_aes(size=...) to default_aes(linewidth=...) #28

ammar-gla opened this issue Apr 26, 2023 · 0 comments

Comments

@ammar-gla
Copy link

I don't know if anyone is maintaining this package, but due to a change in ggplot2 (tidyverse/ggplot2#3672 (comment)), the line-based geoms should reference linewidth instead of size, such as the below.

GeomGLALine <- ggplot2::ggproto("GeomGLALine", ggplot2::GeomLine, default_aes = ggplot2::aes( size= 3 * mm_to_pt)
to
GeomGLALine <- ggplot2::ggproto("GeomGLALine", ggplot2::GeomLine, default_aes = ggplot2::aes( linewidth = 3 * mm_to_pt)

Otherwise there is an issue where setting a static size value to a line outside the aes() field is completely ignored (i.e. when I do not want it to vary by group).

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

No branches or pull requests

1 participant