Closed emilsjoerup closed 3 years ago
Merging #59 (5f3619b) into master (4e66786) will increase coverage by
1.27%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #59 +/- ##
==========================================
+ Coverage 57.50% 58.78% +1.27%
==========================================
Files 24 24
Lines 5782 5782
==========================================
+ Hits 3325 3399 +74
+ Misses 2457 2383 -74
Impacted Files | Coverage Δ | |
---|---|---|
R/dataHandling.R | 53.27% <ø> (+1.63%) |
:arrow_up: |
R/internalDataHandling.R | 85.65% <0.00%> (+23.91%) |
: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 4e66786...5f3619b. Read the comment docs.
Hi Kris,
I thought I had already made this change but it was not done properly
I have changed the lagQuotes variable default to 0 as that is what most users would expect and what should be done with newer datasets.
I also added a unit test for backwards forwards matching algorithm.