r-devel / r-project-sprint-2023

Material for the R project sprint
https://contributor.r-project.org/r-project-sprint-2023/
17 stars 3 forks source link

Refactor `parallel` cluster code to allow packages to provide alternate/custom backends #61

Closed gmbecker closed 1 year ago

gmbecker commented 1 year ago

Endorsed by Luke.

Currently (AFAIK) functionality is spread across the parallel package and some hardcoded links to functionality in the snow package.

gmbecker commented 1 year ago

Functionality needed:

cluster object which can:

gmbecker commented 1 year ago

@shikokuchuo I believe you'er the right one here. Last I heard you had something that conceptualloy works but may need some previously internal generics in parallel exported. Is that correct?

shikokuchuo commented 1 year ago

Conceptually yes, actually not yet. The unexported generics I discussed with Luke this morning and it shouldn't be an issue for them to be exported (just a historical artefact that they weren't). Once I have something that's working I'll detail what exactly needs to be changed.

gmbecker commented 1 year ago

Ultimately its up to @ltierney but my 2c is to remember that parallel is a very well established existing system. Pure extensions (such as exporting things that weren't before, or adding a package backend registration system eventually) may be doable, but any backwards incompatible changes would likely need really strong, general justifications to be considered.

shikokuchuo commented 1 year ago

Bugzilla issue 18587 filed. https://bugs.r-project.org/show_bug.cgi?id=18587

ltierney commented 1 year ago

Thanks! Let's continue working on this on bugzilla.