shiny icon indicating copy to clipboard operation
shiny copied to clipboard

removing the priority of the `ragg` package in the `shiny:: startPNG` - crashing the R session

Open Polkas opened this issue 2 years ago • 2 comments

linked to https://github.com/r-lib/ragg/issues/116 very important it seems to not be a problem of arm64 only.

Please consider removal of the priority of the ragg package in the shiny::startPNG. The ragg package is causing a hard crash of the R session and unfortunately maintainer seems to NOT have time for fixes.

In my opinion the core R grDevices seems to be a much better choice, at least regarding stability.

Polkas avatar Sep 19 '22 10:09 Polkas

You can set options(shiny.useragg = FALSE) to revert to the old behavior

cpsievert avatar Sep 19 '22 14:09 cpsievert

Yes, I know. Still it is not obvious to do so each time or putting it to .Rprofile. I feel the default shiny should be as stable as possible. It is only my feeling, feel free to close the issue.

Polkas avatar Sep 19 '22 14:09 Polkas

@Polkas Are you saying this is causing a crash with valid inputs? Or only when you somehow end up with a plot whose width or height is NULL?

jcheng5 avatar Sep 29 '22 00:09 jcheng5

Sorry for a slow response as I was on vacation. First of all valid or not valid it should not matter as It hard crash the R session. The ragg package is simply unstable at the moment. The problem appeared in the custom shiny module. I think it is related to https://github.com/rstudio/shiny/issues/3637 so the height and width which are inputs are for a moment NULLs (first empty input cycle). Thanks for considering this problem.

Polkas avatar Oct 06 '22 01:10 Polkas

I can reproduce the crash when using a NULL value for height or width. For sure we should validate these arguments in our function (and of course it would be great if ragg would check them too instead of segfaulting).

jcheng5 avatar Oct 06 '22 02:10 jcheng5