search

sanity / quickml

A fast and easy to use decision tree learner in java
http://quickml.org/
GNU Lesser General Public License v3.0
231 stars 54 forks source link

DecisionOldOldTreeBuilderTest.mockTreeTest is flaky #152

Open flakycov opened 7 years ago

flakycov commented 7 years ago

DecisionOldOldTreeBuilderTest.mockTreeTest fails for us on commit 2b348d5 but did not fail on the corresponding run on Travis.

Our tool confirms this test to be flaky. The stack trace is below. We set up our build environment to be as close as possible to your Travis build environment, using Ubuntu 12.04.5 LTS, Java 1.7.0_76 and Maven 3.3.9.

Tests run: 106, Failures: 1, Errors: 0, Skipped: 6, Time elapsed: 104.695 sec <<< FAILURE! - in TestSuite
mockTreeTest on mockTreeTest(quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest)(quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest)  Time elapsed: 5.922 sec  <<< FAILURE!
java.lang.AssertionError: attribute: ecp.  branch.getProbabilityOfTrueChild(): 0.9
    at org.junit.Assert.fail(Assert.java:88)
    at org.junit.Assert.assertTrue(Assert.java:41)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest$Conditions.satisfiesBranchConditions(DecisionOldOldTreeBuilderTest.java:159)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest$Conditions.satisfiesConditions(DecisionOldOldTreeBuilderTest.java:152)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest.recurseTree(DecisionOldOldTreeBuilderTest.java:127)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest.recurseTree(DecisionOldOldTreeBuilderTest.java:130)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest.recurseTree(DecisionOldOldTreeBuilderTest.java:130)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest.recurseTree(DecisionOldOldTreeBuilderTest.java:129)
    at quickml.supervised.tree.decisionTree.DecisionOldOldTreeBuilderTest.mockTreeTest(DecisionOldOldTreeBuilderTest.java:91)