Closed azimov closed 1 year ago
Merging #1003 (04dcf35) into develop (09bde2d) will increase coverage by
5.23%
. The diff coverage is88.87%
.:exclamation: Current head 04dcf35 differs from pull request most recent head 3eaa18c. Consider uploading reports for the commit 3eaa18c to get more accurate results
@@ Coverage Diff @@
## develop #1003 +/- ##
===========================================
+ Coverage 82.07% 87.31% +5.23%
===========================================
Files 16 19 +3
Lines 3281 3919 +638
===========================================
+ Hits 2693 3422 +729
+ Misses 588 497 -91
Impacted Files | Coverage Δ | |
---|---|---|
R/Shiny.R | 26.42% <32.74%> (-21.11%) |
:arrow_down: |
R/Shared.R | 66.66% <66.66%> (ø) |
|
R/Incremental.R | 93.08% <78.94%> (+12.96%) |
:arrow_up: |
R/CohortRelationship.R | 79.01% <79.01%> (ø) |
|
R/DataSourceInformation.R | 81.96% <81.96%> (ø) |
|
R/CohortCharacterizationDiagnostics.R | 82.08% <82.60%> (+6.19%) |
:arrow_up: |
R/ResultsDataModel.R | 71.65% <86.56%> (-2.02%) |
:arrow_down: |
R/RunDiagnostics.R | 89.73% <88.80%> (+9.70%) |
:arrow_up: |
R/Private.R | 85.46% <92.64%> (+3.24%) |
:arrow_up: |
R/ConceptIds.R | 97.29% <95.12%> (-0.35%) |
:arrow_down: |
... and 17 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
basically just lets temporalCovariateSettings to be a list where there first entry is required to be the default covariate settings object