-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nested remote Biocparallel #1
Comments
Hey and congrats to the first issue posted here :) Correct, the problem is the same as you observed with doFuture; the ExampleHere is a minimal example showing what the problem currently is and the manual workaround: > library("BiocParallel.FutureParam")
> register(FutureParam())
> plan(multisession)
# The BiocParallel back-end that master will use
> bpparam()
class: FutureParam
bpisup: TRUE; bpnworkers: 4; bptasks: 0; bpjobname: BPJOB
bplog: FALSE; bpthreshold: INFO; bpstopOnError: TRUE
bptimeout: 2592000; bpprogressbar: FALSE
bplogdir: NA
# The BiocParallel back-end that a worker will use
> bp %<-% bpparam()
> bp
class: MulticoreParam
bpisup: FALSE; bpnworkers: 1; bptasks: 0; bpjobname: BPJOB
bplog: FALSE; bpthreshold: INFO; bpstopOnError: TRUE
bptimeout: 2592000; bpprogressbar: FALSE
bpRNGseed:
bplogdir: NA
bpresultdir: NA
cluster type: FORK Manual workaround: > bp %<-% { register(FutureParam()); bpparam() }
class: FutureParam
bpisup: TRUE; bpnworkers: 4; bptasks: 0; bpjobname: BPJOB
bplog: FALSE; bpthreshold: INFO; bpstopOnError: TRUE
bptimeout: 2592000; bpprogressbar: FALSE
bplogdir: NA ActionMy plan is to add a mechanism to the future package allowing doFuture, BiocParallel.FutureParam, (your favorite higher-level future orchestration API here) to add an "onEntry" hook function that will be called whenever |
Just to confirm, after fiddling around a little to understand what you meant, this code worked :) This is an example using DESeq2, an RNAseq analysis package that is very often used. Sweeet!
|
It is me again :)
Does the nesting of the futures also work with BiocParallel.FutureParam like doFuture and Future does?
Does this locally:
Translate to this when working with a remote machine:
With the same syntax of %->% to "peel off" futures?
The text was updated successfully, but these errors were encountered: