Skip to content

Commit

Permalink
chore(detectors): Rename outlier detector base class. (#203)
Browse files Browse the repository at this point in the history
  • Loading branch information
csanden committed Jan 12, 2018
1 parent ed424ab commit ff8f081
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

package com.netflix.kayenta.judge

import com.netflix.kayenta.judge.detectors.OutlierDetector
import com.netflix.kayenta.judge.detectors.BaseOutlierDetector


object Transforms {
Expand Down Expand Up @@ -53,7 +53,7 @@ object Transforms {
* @param data
* @param detector
*/
def removeOutliers(data: Array[Double], detector: OutlierDetector): Array[Double] ={
def removeOutliers(data: Array[Double], detector: BaseOutlierDetector): Array[Double] ={
val outliers = detector.detect(data)
data.zip(outliers).collect{case (v, false) => v}
}
Expand All @@ -64,7 +64,7 @@ object Transforms {
* @param detector
* @return
*/
def removeOutliers(metric: Metric, detector: OutlierDetector): Metric = {
def removeOutliers(metric: Metric, detector: BaseOutlierDetector): Metric = {
metric.copy(values = removeOutliers(metric.values, detector))
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

package com.netflix.kayenta.judge.detectors

abstract class OutlierDetector{
abstract class BaseOutlierDetector{

/**
* Determine which data points are outliers
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import com.netflix.kayenta.judge.stats.DescriptiveStatistics.percentile
* Reference: https://en.wikipedia.org/wiki/Outlier#Tukey.27s_test
* Note: To reduce sensitivity, take the max of the IQR or the 99th percentile
*/
class IQRDetector(factor: Double = 1.5, reduceSensitivity: Boolean = false) extends OutlierDetector {
class IQRDetector(factor: Double = 1.5, reduceSensitivity: Boolean = false) extends BaseOutlierDetector {

require(factor > 0.0, "factor must be greater than zero")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import org.apache.commons.math3.stat.StatUtils
* Values which are greater than or less than k standard deviations from the mean are considered outliers
* Reference: https://en.wikipedia.org/wiki/68%E2%80%9395%E2%80%9399.7_rule
*/
class KSigmaDetector(k: Double = 3.0) extends OutlierDetector{
class KSigmaDetector(k: Double = 3.0) extends BaseOutlierDetector{

require(k > 0.0, "k must be greater than zero")

Expand Down

0 comments on commit ff8f081

Please sign in to comment.