Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/21465#discussion_r239240113
--- Diff: python/pyspark/ml/regression.py ---
@@ -705,12 +710,59 @@ def getNumTrees(self):
return self.getOrDefault(self.numTrees)
-class GBTParams(TreeEnsembleParams):
+class GBTParams(TreeEnsembleParams, HasMaxIter, HasStepSize,
HasValidationIndicatorCol):
"""
Private class to track supported GBT params.
"""
+
+ stepSize = Param(Params._dummy(), "stepSize",
+ "Step size (a.k.a. learning rate) in interval (0, 1]
for shrinking " +
+ "the contribution of each estimator.",
+ typeConverter=TypeConverters.toFloat)
+
+ validationTol = Param(Params._dummy(), "validationTol",
+ "Threshold for stopping early when fit with
validation is used. " +
+ "If the error rate on the validation input
changes by less than the " +
+ "validationTol, then learning will stop early
(before `maxIter`). " +
+ "This parameter is ignored when fit without
validation is used.",
+ typeConverter=TypeConverters.toFloat)
+
+ @since("3.0.0")
+ def setValidationTol(self, value):
--- End diff --
It seems scala does not have this API right? If not then let's remove it
here for now
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
