Median diff improved#6
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6 +/- ##
==========================================
- Coverage 100% 99.39% -0.61%
==========================================
Files 2 2
Lines 144 166 +22
Branches 13 19 +6
==========================================
+ Hits 144 165 +21
- Misses 0 1 +1
Continue to review full report at Codecov.
|
|
Hi MedianDifferencing is meant to remove numbers where the difference between consecutive numbers is higher than the median. If 2 numbers say 58, 56 are next to each other, it would not be an outlier under this method. You'll want to use the other outlier method MAD that doesnt care about order. |
|
Okay, but the number just after the outlier also get deleted, since the difference for it will also be higher with outlier, which is the reason of data loss |
The last median diff function was not working in the following cases:
1: When the outlier is the first element in array
2: When there are two or more consecutive outliers within the same range, like 58, 54.
3: Also it used to remove the element following the outlier too, due to which there was data loss, which has also been fixed