passing expression to r() about callr

[gaborcsardi]

I think functions are cleaner. I also wanted to avoid doing NSE, although your quote way does that. Still, imo a function is simpler for people than a quote.

It is still not optimal, because we need to reset their environment manually, so you cannot pass a closure for example.

[gaborcsardi]

I don't mind handling expressions, do you want to submit a PR? I think we can just put the expressions into a function, and then everything just works.

[gaborcsardi]

I think it is cleaner to use functions, with the environment reset.

[kforner]

I don't understand why ? Could you please give me an example ?
My current work-around is:

#' execute some R code via the R executable 
#' 
#' @param code 	a language object. 
#' @param ... 	for callr::r 
#' 
#' @return the code evaluation  
#' @export 
run_r_code <- function(code, ...) {
   fun <- function() {}  
   body(fun) <- code   
   callr::r(fun, ...)
}

[gaborcsardi]

How do you pass an "argument" to code?

[kforner]

I can't. And I just understand now why you use functions. I had to use bquote on my expressions to pass my local vars. Makes perfect sense.

The point was to run simple commands such as run_r_code(quote(Sys.getwd())
It can be improved with : body(fun) <- substitute(code)
to be: run_r_code(Sys.getwd()).
I agree that this is only syntactic sugar, for me it felt more natural than giving a function.

[gaborcsardi]

I think it can be done with expressions, but it is just way simpler with a function, and function arguments. And it is not so much more to write function() expr than expr.

[kforner]

Definitely. As I said I missed the args passing problem. And anyway I can use my wrapper so everybody is happy ;)

Thanks again for callr. Did you know that you can tremendously speed up R CMD check using r_cmd() by setting repos to "" ? It skips the utils::available.packages network download...

[gaborcsardi]

Thanks again for callr. Did you know that you can tremendously speed up R CMD check using r_cmd() by setting repos to "" ? It skips the utils::available.packages network download...

That's a good idea.

Btw. I want to "fix" the available.packages caching problem, so that it is kept across sessions and updates properly: r-lib/crancache#14

[kforner]

Btw. I want to "fix" the available.packages caching problem, so that it is kept across sessions and updates properly: r-lib/crancache#14

Very interesting indeed !
I was not aware of this r_hub project. Is it going well ? Who is driving it ?

[gaborcsardi]

Rhub in general? It is an R Consortium project.

…

On 14 May 2017 11:30, "Karl Forner" ***@***.***> wrote: Btw. I want to "fix" the available.packages caching problem, so that it is kept across sessions and updates properly: r-lib/crancache#14 <r-lib/crancache#14> Very interesting indeed ! I was not aware of this r_hub project. Is it going well ? Who is driving it ? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#10 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAoTQHY1F4Yw5rWzYVeQFIbgh0r0ckq5ks5r5te6gaJpZM4JINMk> .