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

Update documentation for agg_png with ggsave #66

Closed
jjcrumpler3 opened this issue Jan 8, 2021 · 1 comment
Closed

Update documentation for agg_png with ggsave #66

jjcrumpler3 opened this issue Jan 8, 2021 · 1 comment

Comments

@jjcrumpler3
Copy link

First time I've reported an issue, so please bare with me if I leave anything out. The protocol for saving a plot with ggsave with device = agg_png seems to have changed, but not clearly communicated within documentation.

previous example may be something like

ggsave("mtcars.png", device = agg_png, res = 400)

However, the working method now is

ggsave("mtcars.png", device = agg_png(res = 400))

I like the change, but it does not seem to be documented anywhere, at least nowhere that I noticed.

@thomasp85
Copy link
Member

This is not correct. The right way to use ragg with ggsave() is the first mentioned approach... There is, however, some unfortunate interaction between the res argument in ragg_png() and the dpi argument in ggsave()

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

2 participants