Closed ChrisJones687 closed 3 years ago
closes #101
Merging #105 (4e95c37) into master (c3903d7) will increase coverage by
0.17%
. The diff coverage isn/a
.:exclamation: Current head 4e95c37 differs from pull request most recent head d13e7b1. Consider uploading reports for the commit d13e7b1 to get more accurate results
@@ Coverage Diff @@
## master #105 +/- ##
==========================================
+ Coverage 75.25% 75.43% +0.17%
==========================================
Files 40 40
Lines 4825 4852 +27
==========================================
+ Hits 3631 3660 +29
+ Misses 1194 1192 -2
Impacted Files | Coverage Δ | |
---|---|---|
R/configuration.R | 84.17% <0.00%> (-0.77%) |
:arrow_down: |
R/pops.r | 100.00% <0.00%> (ø) |
|
R/auto_manage.R | 0.00% <0.00%> (ø) |
|
R/validate.R | 99.41% <0.00%> (+0.01%) |
:arrow_up: |
R/pops_multirun.R | 94.44% <0.00%> (+0.09%) |
:arrow_up: |
R/calibrate.R | 91.84% <0.00%> (+0.21%) |
:arrow_up: |
R/quantity_allocation_disagreement.R | 92.30% <0.00%> (+5.98%) |
: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 c3903d7...d13e7b1. Read the comment docs.
Seems to have actually worked this time. Not sure what happened last time.
@wenzeslaus thoughts on this format for adding features to the readme for the pops repositories. I may have missed some features and parameters on this pass but am hoping that something like this helps clarify which features are linked to which parameters. Not sure if that is clear or if the parameters need to be more explicitly spelled out somehow.