remove unnecessary doc

apache · May 17, 2015 · 413c463 · 413c463
1 parent 4159f35
commit 413c463
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 28 deletions.
diff --git a/python/pyspark/ml/pipeline.py b/python/pyspark/ml/pipeline.py
@@ -196,17 +196,9 @@ def _fit(self, dataset):
         return PipelineModel(transformers)
 
     def copy(self, extra={}):
-        """
-        Creates a copy of this instance with a randomly generated uid
-        and some extra params. This copies each of the component
-        stages, creates a deep copy of the embedded paramMap, and
-        copies the embedded and extra parameters over.
-        :param extra: Extra parameters to copy to the new instance
-        :return: Copy of this instance
-        """
-        paramMap = self.extractParamMap(extra)
-        stages = [stage.copy(extra) for stage in paramMap[self.stages]]
-        return Pipeline().setStages(stages)
+        that = Params.copy(self, extra)
+        stages = [stage.copy(extra) for stage in that.getStages()]
+        return that.setStages(stages)
 
 
 @inherit_doc
@@ -225,14 +217,6 @@ def _transform(self, dataset):
         return dataset
 
     def copy(self, extra={}):
-        """
-        Creates a copy of this instance with a randomly generated uid
-        and some extra params. This copies each of the component
-        stages, creates a deep copy of the embedded paramMap, and
-        copies the embedded and extra parameters over.
-        :param extra: Extra parameters to copy to the new instance
-        :return: Copy of this instance
-        """
         stages = [stage.copy(extra) for stage in self.stages]
         return PipelineModel(stages)
 

diff --git a/python/pyspark/ml/tuning.py b/python/pyspark/ml/tuning.py
@@ -229,15 +229,6 @@ def _fit(self, dataset):
         return CrossValidatorModel(bestModel)
 
     def copy(self, extra={}):
-        """
-        Creates a copy of this instance with a randomly generated uid
-        and some extra params. This copies the underlying estimator,
-        evaluator, and estimatorParamMap, creates a deep copy of the
-        embedded paramMap, and copies the embedded and extra parameters
-        over.
-        :param extra: Extra parameters to copy to the new instance
-        :return: Copy of this instance
-        """
         newCV = Params.copy(self, extra)
         if self.isSet(self.estimator):
             newCV.setEstimator(self.getEstimator().copy(extra))