Closed azimov closed 1 year ago
Merging #100 (8018446) into main (30fe75d) will not change coverage. The diff coverage is
100.00%
.:exclamation: Current head 8018446 differs from pull request most recent head 6c8175e. Consider uploading reports for the commit 6c8175e to get more accurate results
@@ Coverage Diff @@
## main #100 +/- ##
=======================================
Coverage 98.54% 98.54%
=======================================
Files 12 12
Lines 1370 1370
=======================================
Hits 1350 1350
Misses 20 20
Impacted Files | Coverage Δ | |
---|---|---|
R/SubsetDefinitions.R | 98.48% <100.00%> (ø) |
target pairs will overflow when the id is something like
1557272 * 1000 + 1
which will happen for cohorts where we do something likeconceptId * 1000
Resolves #101