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

    https://github.com/apache/spark/pull/20146#discussion_r159577911
  
    --- Diff: R/pkg/tests/fulltests/test_mllib_classification.R ---
    @@ -313,7 +313,7 @@ test_that("spark.mlp", {
       # Test predict method
       mlpTestDF <- df
       mlpPredictions <- collect(select(predict(model, mlpTestDF), 
"prediction"))
    -  expect_equal(head(mlpPredictions$prediction, 6), c("1.0", "0.0", "0.0", 
"0.0", "0.0", "0.0"))
    +  expect_equal(head(mlpPredictions$prediction, 6), c("0.0", "1.0", "1.0", 
"1.0", "1.0", "1.0"))
    --- End diff --
    
    This is due to the change of how we sort string labels with same frequency 
under the setting of frequencyDesc/Asc.


---

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

Reply via email to