Repository: spark
Updated Branches:
  refs/heads/branch-2.0 53c09f065 -> 1346f3cd6


[SPARK-15444][PYSPARK][ML][HOTFIX] Default value mismatch of param 
linkPredictionCol for GeneralizedLinearRegression

## What changes were proposed in this pull request?

Default value mismatch of param linkPredictionCol for 
GeneralizedLinearRegression between PySpark and Scala. That is because default 
value conflict between #13106 and #13129. This causes ml.tests failed.

## How was this patch tested?
Existing tests.

Author: Liang-Chi Hsieh <sim...@tw.ibm.com>

Closes #13220 from viirya/hotfix-regresstion.

(cherry picked from commit 4e739331187f2acdd84a5e65857edb62e58a0f8f)
Signed-off-by: Nick Pentreath <ni...@za.ibm.com>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/1346f3cd
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/1346f3cd
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/1346f3cd

Branch: refs/heads/branch-2.0
Commit: 1346f3cd6cf78c940f646bb2b808ae3b22f936b3
Parents: 53c09f0
Author: Liang-Chi Hsieh <sim...@tw.ibm.com>
Authored: Fri May 20 13:40:13 2016 +0200
Committer: Nick Pentreath <ni...@za.ibm.com>
Committed: Fri May 20 13:40:27 2016 +0200

----------------------------------------------------------------------
 python/pyspark/ml/regression.py | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/1346f3cd/python/pyspark/ml/regression.py
----------------------------------------------------------------------
diff --git a/python/pyspark/ml/regression.py b/python/pyspark/ml/regression.py
index 25640b1..e21dd83 100644
--- a/python/pyspark/ml/regression.py
+++ b/python/pyspark/ml/regression.py
@@ -1303,17 +1303,16 @@ class GeneralizedLinearRegression(JavaEstimator, 
HasLabelCol, HasFeaturesCol, Ha
     @keyword_only
     def __init__(self, labelCol="label", featuresCol="features", 
predictionCol="prediction",
                  family="gaussian", link=None, fitIntercept=True, maxIter=25, 
tol=1e-6,
-                 regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=""):
+                 regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=None):
         """
         __init__(self, labelCol="label", featuresCol="features", 
predictionCol="prediction", \
                  family="gaussian", link=None, fitIntercept=True, maxIter=25, 
tol=1e-6, \
-                 regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol="")
+                 regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=None)
         """
         super(GeneralizedLinearRegression, self).__init__()
         self._java_obj = self._new_java_obj(
             "org.apache.spark.ml.regression.GeneralizedLinearRegression", 
self.uid)
-        self._setDefault(family="gaussian", maxIter=25, tol=1e-6, 
regParam=0.0, solver="irls",
-                         linkPredictionCol="")
+        self._setDefault(family="gaussian", maxIter=25, tol=1e-6, 
regParam=0.0, solver="irls")
         kwargs = self.__init__._input_kwargs
         self.setParams(**kwargs)
 
@@ -1321,11 +1320,11 @@ class GeneralizedLinearRegression(JavaEstimator, 
HasLabelCol, HasFeaturesCol, Ha
     @since("2.0.0")
     def setParams(self, labelCol="label", featuresCol="features", 
predictionCol="prediction",
                   family="gaussian", link=None, fitIntercept=True, maxIter=25, 
tol=1e-6,
-                  regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=""):
+                  regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=None):
         """
         setParams(self, labelCol="label", featuresCol="features", 
predictionCol="prediction", \
                   family="gaussian", link=None, fitIntercept=True, maxIter=25, 
tol=1e-6, \
-                  regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol="")
+                  regParam=0.0, weightCol=None, solver="irls", 
linkPredictionCol=None)
         Sets params for generalized linear regression.
         """
         kwargs = self.setParams._input_kwargs


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

Reply via email to