Github user WeichenXu123 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19666#discussion_r149567340
  
    --- Diff: 
mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala ---
    @@ -631,6 +614,42 @@ class RandomForestSuite extends SparkFunSuite with 
MLlibTestSparkContext {
         val expected = Map(0 -> 1.0 / 3.0, 2 -> 2.0 / 3.0)
         assert(mapToVec(map.toMap) ~== mapToVec(expected) relTol 0.01)
       }
    +
    +  test("traverseUnorderedSplits") {
    +
    --- End diff --
    
    So how to test all possible splits to make sure the generated splits are 
all correct ? If tree generated, only best split is remained.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to