walkAST() gives error with S7 object.

[egenn]

Hi - This is similar to #28, #27, but with S7 objects.
Passing an S7 object to a future.apply::future_lapply() call throws error
Error: Cannot walk expression. Unknown object type ‘object’.
Runs with future.globals = FALSE.

Create S7 object:

> Dat <- S7::new_class(
+   name = "Dat",
+   properties = list(
+     value = S7::class_numeric
+   )
+ )
> obj <- Dat(value = 1.2)

Minimal example:

> future.apply::future_lapply(1L, function(i, x) class(x), x = obj)
Error: Cannot walk expression. Unknown object type ‘object’
10: stop(msg, call. = FALSE)
9: walkAST(expr[[cc]], atomic = atomic, name = name, call = call,
       pairlist = pairlist, substitute = FALSE)
8: walkAST(expr, call = tweakFutureAssignmentCall)
7: tweak(expr)
6: findGlobals(expr, envir = envir, ..., method = method, tweak = tweak,
       substitute = FALSE, unlist = unlist)
5: globalsOf(expr, envir = envir, substitute = FALSE, tweak = tweak,
       locals = locals, dotdotdot = "return", method = globals.method,
       unlist = TRUE, mustExist = mustExist, recursive = TRUE)
4: getGlobalsAndPackages(expr, envir = envir, globals = globals)
3: getGlobalsAndPackagesXApply(FUN = FUN, args = args, MoreArgs = MoreArgs,
       envir = envir, future.globals = future.globals, future.packages = future.packages,
       debug = debug)
2: future_xapply(FUN = FUN, nX = nX, chunk_args = X, args = list(...),
       get_chunk = `chunkWith[[`, expr = expr, envir = envir, future.envir = future.envir,
       future.globals = future.globals, future.packages = future.packages,
       future.scheduling = future.scheduling, future.chunk.size = future.chunk.size,
       future.stdout = future.stdout, future.conditions = future.conditions,
       future.seed = future.seed, future.label = future.label, fcn_name = fcn_name,
       args_name = args_name, debug = debug)
1: future.apply::future_lapply(1L, function(idx, x) class(x), x = obj)

Works with globals disabled:

> future.apply::future_lapply(1L, function(i, x) class(x), x = obj, future.globals = FALSE)
[[1]]
[1] "Dat"       "S7_object"

[HenrikBengtsson]

Thanks. I think a minimal reproducible example of this is:

A <- S7::new_class(name = "A")
obj <- A()
globals::walkAST(obj, call = identity)
#> Error: Cannot walk expression. Unknown object type 'object'

[HenrikBengtsson]

I actually found a smaller example:

obj <- S7::S7_object()
globals::walkAST(obj, call = identity)
#> Error: Cannot walk expression. Unknown object type 'object'

[egenn]

globals::walkAST(S7::S7_object())
#> Error: Cannot walk expression. Unknown object type ‘object’

[HenrikBengtsson]

Got a minimal reproducible example that doesn't bring in non-base-R packages thanks to RConsortium/S7#524 (comment);

> obj <- asS3(methods::getClass("S4")@prototype, complete = FALSE)
> globals::walkAST(obj)
Error: Cannot walk expression. Unknown object type 'object'

Now I can add this as a unit test, and then fix it.

[HenrikBengtsson]

This has been fixed in the next release, e.g.

> globals::walkAST(S7::S7_object())
<S7_object>

Until submitted to CRAN, it's available as:

remotes::install_github("futureverse/globals", ref = "develop")

Thanks for reporting.