futureverse / futureverse/future.apply

Single future.control argument rather than multiple, individual future.* arguments

Open
#27 9 comments 0 reactions 0 assignees View on GitHub
feature request
Dominant language
R
Stars
218
Forks
20
Avg merge
2d 9h
Merged PRs (30d)
1

Description

@mllg [wrote](https://github.com/HenrikBengtsson/future.apply/issues/25#issuecomment-420955847):

> Have you considered to introduce a control object (in the fashion of passing a `rpart.control`-object to `rpart()`)? A `future.control`-object could bundle all arguments. Would be good for the overview (also in the documentation) and ...

Yes, I've been having internal (as in lots of inner voices ;)) debates about this and it's been discussed with other in the past. I'm not opposed to it. The main reason I've stayed away from it is that we have to deside exactly how the control elements should be controlled.

> ... you would avoid inconsistencies like https://github.com/HenrikBengtsson/future.apply/issues/26.

You're meaning in the sense that the future.apply package and other similar high-level packages (e.g. furrr) won't have to know about future-specific arguments and can just pass whatever down to the future package? That's a nice side effect I haven't thought of before.

However, not all elements in `future.control` should be passed down to the future package. For instance, `scheduling` and `chunk.size` are higher level properties. If so, do they belong to a `future.control` argument or should they be separate?

So several things to think of. Thanks for bringing this up.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.