[mlpack-git] master: Fix test condition. (c525694)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Wed Dec 23 12:24:17 EST 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/734a0b154c3835563709cc0d5e30602b58674e52...c5256940687b97b0d140009019b31a769a260a66

>---------------------------------------------------------------

commit c5256940687b97b0d140009019b31a769a260a66
Author: ryan <ryan at ratml.org>
Date:   Wed Dec 23 12:23:49 2015 -0500

    Fix test condition.
    
    We aren't guaranteed that bestGain > secondBestGain, just that bestGain >=
    secondBestGain.


>---------------------------------------------------------------

c5256940687b97b0d140009019b31a769a260a66
 src/mlpack/tests/hoeffding_tree_test.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mlpack/tests/hoeffding_tree_test.cpp b/src/mlpack/tests/hoeffding_tree_test.cpp
index 584bf27..d3129da 100644
--- a/src/mlpack/tests/hoeffding_tree_test.cpp
+++ b/src/mlpack/tests/hoeffding_tree_test.cpp
@@ -704,7 +704,7 @@ BOOST_AUTO_TEST_CASE(BinaryNumericSplitSimpleFourClassSplitTest)
     double bestGain, secondBestGain;
     split.EvaluateFitnessFunction(bestGain, secondBestGain);
     BOOST_REQUIRE_CLOSE(bestGain, 0.25, 1e-5);
-    BOOST_REQUIRE_GT(bestGain, secondBestGain);
+    BOOST_REQUIRE_GE(bestGain, secondBestGain);
   }
 
   // Now, when we ask it to split, ensure that the split value is reasonable.



More information about the mlpack-git mailing list