Error when making generator a method of an R6 class about coro HOT 5 CLOSED

In the meantime you can wrap the generators like this @dfalbel

R6::R6Class(
  "generator",
  private = list(
    generators = list(x = coro::generator(function() {
      for (i in 1:10)
        yield(i)
    }))
  ),
  public = list(
    x = function() private$generators$x()
  )
)

from coro.

Thanks for your input @wch!
I've filed an issue for I() at r-lib/R6#222.

Closing this since it's not a coro issue.

from coro.

Maybe there should be a way of preventing R6 from treating a function as a method? For example functions wrapped in I(). And maybe functions with a custom class should automatically get that treatment?

Relatedly, maybe R6Class() should issue a warning when methods don't inherit from the current environment? Changing their environments in the background seems a bit unsafe.

What do you think @wch?

from coro.

Thanks! That works great for me!

from coro.

With R6Class, all functions that are part of public or private are assumed to be methods, and their environment is modified when the class is instantiated.

R6 does have the notion of members function that are not methods, though. They must be assigned either in initialize() or after the object is created. So you could so something like this:

num_generator <- coro::generator(function() {
  for (i in 1:10)
    yield(i)
})

Generator <- R6::R6Class(
  "Generator",
  public = list(
    initialize = function() {
      self$x = num_generator
    },
    x = NULL
  )
)

(You could also call coro::generator(function() ....) inside of initialize(), but that would create a new factory for each instance, which is less efficient.)

The difference between methods and non-method functions is recognized when the object is cloned; in the cloned object, methods will have their environment changed (so they can find the correct self and private), but non-method functions will not have their environment changed.

@lionel-, the idea of using I() to mark non-method functions is interesting. Want to file an issue on R6 so it can be discussed there? Or maybe the current behavior just needs to be better documented.

from coro.