honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ML] Disable invalid assertion (#50988) 

Backport #50986.
This commit is contained in:
Tom Veasey 2020-01-14 17:35:00 +00:00 committed by GitHub
parent ca9ca68cbe
commit de5713fa4b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
x-pack/plugin/ml/qa/native-multi-node-tests/src/test/java/org/elasticsearch/xpack/ml/integration/ClassificationIT.java
View File

@ -5,7 +5,8 @@
*/
package org.elasticsearch.xpack.ml.integration;
import com.google.common.collect.Ordering;
// Pending fix
//import com.google.common.collect.Ordering;
import org.elasticsearch.ElasticsearchStatusException;
import org.elasticsearch.action.admin.indices.get.GetIndexAction;
import org.elasticsearch.action.admin.indices.get.GetIndexRequest;
@ -592,7 +593,9 @@ public class ClassificationIT extends MlNativeDataFrameAnalyticsIntegTestCase {
// Assert that all the class probabilities lie within [0, 1] interval.
classProbabilities.forEach(p -> assertThat(p, allOf(greaterThanOrEqualTo(0.0), lessThanOrEqualTo(1.0))));
// Assert that the top classes are listed in the order of decreasing probabilities.
assertThat(Ordering.natural().reverse().isOrdered(classProbabilities), is(true));
// This is not true after https://github.com/elastic/ml-cpp/pull/926. I'll fix and re-enable
// once that change is merged.
//assertThat(Ordering.natural().reverse().isOrdered(classProbabilities), is(true));
}
private <T> void assertEvaluation(String dependentVariable, List<T> dependentVariableValues, String predictedClassField) {