Skip to content

Commit 283d94a

Browse files
[ML] fix count ks test aggregator test consistency (#75793) (#75834)
testKsTest_LowerTailedValues is failing due to distribution issues. The random variables in the sparse case are too uniform and causing two-sided alternative tests to not be consistent. closes #74909 Co-authored-by: Benjamin Trent <ben.w.trent@gmail.com>
1 parent 8c441ec commit 283d94a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Diff for: x-pack/plugin/ml/src/test/java/org/elasticsearch/xpack/ml/aggs/kstest/BucketCountKSTestAggregatorTests.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ public class BucketCountKSTestAggregatorTests extends ESTestCase {
3232
new double[] { 40, 60, 20, 30, 30, 10, 10, 10, 10, 10 }
3333
);
3434
private static final MlAggsHelper.DoubleBucketValues LOWER_TAILED_VALUES_SPARSE = new MlAggsHelper.DoubleBucketValues(
35-
new long[] { 4, 6, 2, 3, 3, 2, 1, 1, 1, 1 },
36-
new double[] { 4, 6, 2, 3, 3, 2, 1, 1, 1, 1 }
35+
new long[] { 4, 8, 2, 3, 3, 2, 1, 1, 1, 0 },
36+
new double[] { 4, 8, 2, 3, 3, 2, 1, 1, 1, 0 }
3737
);
3838

3939
private static final MlAggsHelper.DoubleBucketValues UPPER_TAILED_VALUES = new MlAggsHelper.DoubleBucketValues(

0 commit comments

Comments
 (0)