Closed richfitz closed 5 years ago
Merging #9 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #9 +/- ##
======================================
Coverage 100% 100%
======================================
Files 14 16 +2
Lines 849 955 +106
======================================
+ Hits 849 955 +106
Impacted Files | Coverage Δ | |
---|---|---|
R/rrq_controller.R | 100% <100%> (ø) |
:arrow_up: |
R/expression.R | 100% <100%> (ø) |
|
R/worker_run.R | 100% <100%> (ø) |
:arrow_up: |
R/utils.R | 100% <100%> (ø) |
:arrow_up: |
R/worker_messages.R | 100% <100%> (ø) |
:arrow_up: |
R/worker_spawn.R | 100% <100%> (ø) |
:arrow_up: |
R/time.R | 100% <100%> (ø) |
|
R/worker.R | 100% <100%> (ø) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 83f4811...066fc98. Read the comment docs.
This PR removes use of the
context
andqueuer
packages fromrrq
. Doing this will allow us to do two things:lapply
- like interface) in a way that fits better with the new environment creationThe bulk of the new code here was directly copied in from
context
/queuer
with simplifications to remove unused code branches. The code inexpression.R
will change when we do bulk submission, and includes a couple of options that are not (yet) exposed toenqueue
. The progress bar bits are all a bit tedious, but nice for interactive use.