styler issues let runs and tests fail about lucode2

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

Alternatively, since styler is only used via style_file() in lucode2::autoFormat(), and since that function is called nowhere and only going to be invoked by the user, one can move styler to Suggests: and call styler::style_file(), while guarding with rlang::check_installed('styler').
That way styler is not uselessly attached every time lucode2 is attached, and doesn't fail fiddling with its cache for whatever reason.

[pfuehrlich-pik]

Seems like /home/lavinia/.cache/R/R.cache/styler has too restrictive permissions, but that's in Lavinia's home folder (because cronjobs now run using her user account) so I cannot check or change that. Can you have a look @LaviniaBaumstark ?

[orichters]

The strange thing is that this happens not every time. The other AMTs went well, same in the past. Same with the second attempt of the test. No idea what is going on there.

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

Just deactivate the styler caching. It's not used anyway.

[orichters]

@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q: How can this be done?

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

echo 'options(R.cache.enable = FALSE)' >> .Rprofile

[pfuehrlich-pik]

options("styler.cache_name" = NULL)
I'll just put this into our global .Rprofile. We're using styler so infrequently that I don't think small performance improvements justify dealing with weird errors like this. I checked with Lavinia and her file permissions on the folder seem fine.

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

Seems like /home/lavinia/.cache/R/R.cache/styler has too restrictive permissions, but that's in Lavinia's home folder (because cronjobs now run using her user account) […]

So Lavinia's home directory has too restrictive permissions for the cron job running on her account?

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

options("styler.cache_name" = NULL) I'll just put this into our global .Rprofile.

You can't set NULL options. That won't work.

[pfuehrlich-pik]

Setting an option to NULL removes it altogether (as if it was not set at all). In this case it indeed doesn't work because styler checks onLoad whether the option is set. I don't see a way to disable the styler cache via options/envvars so I'll ask Lavinia to create the missing folder manually and hope for the best.

[pfuehrlich-pik]

echo 'options(R.cache.enable = FALSE)' >> .Rprofile

This does disable R.cache in general, not only for styler, so it seems overkill to put this into the global .Rprofile

[0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q]

echo 'options(R.cache.enable = FALSE)' >> .Rprofile

This does disable R.cache in general, not only for styler, so it seems overkill to put this into the global .Rprofile

We don't use any other package that uses R.cache.

[pfuehrlich-pik]

That's a good idea, will do that 👍

[orichters]

Thank you both!