spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From srowen <...@git.apache.org>
Subject [GitHub] spark pull request #19248: [SPARK-22027] Add missing explanation of default ...
Date Fri, 15 Sep 2017 15:53:48 GMT
Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19248#discussion_r139185866
  
    --- Diff: mllib/src/main/scala/org/apache/spark/ml/param/shared/sharedParams.scala ---
    @@ -44,7 +44,7 @@ private[ml] trait HasRegParam extends Params {
     private[ml] trait HasMaxIter extends Params {
     
       /**
    -   * Param for maximum number of iterations (&gt;= 0).
    +   * Param for maximum number of iterations (&gt;= 0). (default = 20)
    --- End diff --
    
    I don't see that this has a global default of 20. Where is that coming from?


---

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


Mime
View raw message