Closed rcannood closed 3 years ago
Merging #29 (b514db1) into master (4f77ead) will decrease coverage by
1.88%
. The diff coverage is70.42%
.:exclamation: Current head b514db1 differs from pull request most recent head 98c9b44. Consider uploading reports for the commit 98c9b44 to get more accurate results
@@ Coverage Diff @@
## master #29 +/- ##
==========================================
- Coverage 68.95% 67.06% -1.89%
==========================================
Files 22 22
Lines 2654 2660 +6
==========================================
- Hits 1830 1784 -46
- Misses 824 876 +52
Impacted Files | Coverage Δ | |
---|---|---|
R/7_experiment.R | 77.08% <ø> (+0.94%) |
:arrow_up: |
R/caching.R | 66.66% <66.66%> (-2.57%) |
:arrow_down: |
R/random_generators.R | 66.66% <66.66%> (ø) |
|
R/4_kinetics.R | 90.58% <70.96%> (-8.50%) |
:arrow_down: |
R/8_convert.R | 66.15% <0.00%> (-15.21%) |
:arrow_down: |
R/0c_backbone_lego.R | 45.61% <0.00%> (+4.09%) |
: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 4f77ead...98c9b44. Read the comment docs.
MINOR CHANGE
.download_cacheable_file()
: Check the return value ofutils::download.file()
, since it is possible that the download will fail with a non-zero status but not an R error.MINOR CHANGE:
kinetics_random_distributions()
: Add function for providing randomised distributions.