spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From WeichenXu123 <...@git.apache.org>
Subject [GitHub] spark pull request #20146: [SPARK-11215][ML] Add multiple columns support to...
Date Thu, 11 Jan 2018 18:41:37 GMT
Github user WeichenXu123 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20146#discussion_r161040131
  
    --- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
    @@ -249,6 +249,16 @@ object ParamValidators {
       def arrayLengthGt[T](lowerBound: Double): Array[T] => Boolean = { (value: Array[T])
=>
         value.length > lowerBound
       }
    +
    +  /** Check if more than one param in a set of exclusive params are set. */
    +  def checkExclusiveParams(model: Params, params: String*): Unit = {
    --- End diff --
    
    @hhbyyh Is this the common helper methods you want for checking in/out single/multiple
columns ?


---

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


Mime
View raw message