Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-17234
ping?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221962428
ping?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221487845
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221487773
**[Test build #59262 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59262/consoleFull)**
for PR 12919 at commit
[`6551fb4`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221487843
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221486047
**[Test build #59262 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59262/consoleFull)**
for PR 12919 at commit
[`6551fb4`](https://gi
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221485607
jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221294271
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabl
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221185913
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221167894
**[Test build #59180 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59180/consoleFull)**
for PR 12919 at commit
[`6551fb4`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221167948
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221167949
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221166703
**[Test build #59180 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59180/consoleFull)**
for PR 12919 at commit
[`6551fb4`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221127854
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221127785
**[Test build #59163 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59163/consoleFull)**
for PR 12919 at commit
[`e498db7`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221127850
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-221126185
**[Test build #59163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59163/consoleFull)**
for PR 12919 at commit
[`e498db7`](https://gi
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r64033754
--- Diff: python/pyspark/ml/regression.py ---
@@ -755,11 +761,35 @@ class TreeEnsembleModels(JavaModel):
"""
@property
+@since(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220424235
**[Test build #58887 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58887/consoleFull)**
for PR 12919 at commit
[`2e2c976`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220424374
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220424378
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220421374
**[Test build #58887 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58887/consoleFull)**
for PR 12919 at commit
[`2e2c976`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220173217
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220173123
**[Test build #58815 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58815/consoleFull)**
for PR 12919 at commit
[`05ec32a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220173214
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-220170612
**[Test build #58815 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58815/consoleFull)**
for PR 12919 at commit
[`05ec32a`](https://gi
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r63369968
--- Diff: python/pyspark/ml/classification.py ---
@@ -772,6 +782,10 @@ class GBTClassifier(JavaEstimator, HasFeaturesCol,
HasLabelCol, HasPredictionCol
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219376949
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219376946
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219376851
**[Test build #58631 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58631/consoleFull)**
for PR 12919 at commit
[`30219a2`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219374674
**[Test build #58631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58631/consoleFull)**
for PR 12919 at commit
[`30219a2`](https://gi
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r63322370
--- Diff: python/pyspark/ml/classification.py ---
@@ -772,6 +782,10 @@ class GBTClassifier(JavaEstimator, HasFeaturesCol,
HasLabelCol, HasPredictionCol
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r63317176
--- Diff: python/pyspark/ml/classification.py ---
@@ -772,6 +782,10 @@ class GBTClassifier(JavaEstimator, HasFeaturesCol,
HasLabelCol, HasPredictionCol
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219311192
So its back to the original proposal for `trees`
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219119112
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219119105
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219118993
**[Test build #58586 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58586/consoleFull)**
for PR 12919 at commit
[`0a3c4c2`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219116489
**[Test build #58586 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58586/consoleFull)**
for PR 12919 at commit
[`0a3c4c2`](https://gi
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-219017485
@holdenk let's finalize this - as I said above, happy to revert to your
original suggestion for `trees`
---
If your project is set up for it, you can reply to this emai
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r63168240
--- Diff: python/pyspark/ml/regression.py ---
@@ -743,6 +743,18 @@ def treeWeights(self):
"""Return the weights for each tree"""
retu
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62730236
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("feat
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62728946
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-218251784
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-218251786
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-218251649
**[Test build #58251 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58251/consoleFull)**
for PR 12919 at commit
[`3cfc996`](https://g
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62727030
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("fea
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-218248738
**[Test build #58251 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58251/consoleFull)**
for PR 12919 at commit
[`3cfc996`](https://gi
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62719285
--- Diff: python/pyspark/ml/regression.py ---
@@ -743,6 +743,18 @@ def treeWeights(self):
"""Return the weights for each tree"""
retu
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62717700
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("feat
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62716733
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("fea
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62714160
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("feat
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62708794
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62707490
--- Diff: python/pyspark/ml/regression.py ---
@@ -738,11 +744,29 @@ class TreeEnsembleModels(JavaModel):
"""
@property
+@si
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62707171
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62670665
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("feat
Github user sethah commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-218024915
I just noticed this, somewhat related: could you add the updates to pyspark
for [#11989](https://github.com/apache/spark/pull/11989/files)? Should be small
:)
---
If y
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217953519
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217953521
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217953367
**[Test build #58157 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58157/consoleFull)**
for PR 12919 at commit
[`17cae03`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217949194
**[Test build #58157 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58157/consoleFull)**
for PR 12919 at commit
[`17cae03`](https://gi
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62517527
--- Diff: python/pyspark/ml/classification.py ---
@@ -857,6 +871,12 @@ def featureImportances(self):
"""
return self._call_java("
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/12919#discussion_r62441488
--- Diff: python/pyspark/ml/regression.py ---
@@ -743,6 +743,18 @@ def treeWeights(self):
"""Return the weights for each tree"""
ret
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217688792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217688781
**[Test build #58085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58085/consoleFull)**
for PR 12919 at commit
[`5f8b0ce`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217688791
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12919#issuecomment-217688416
**[Test build #58085 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58085/consoleFull)**
for PR 12919 at commit
[`5f8b0ce`](https://gi
66 matches
Mail list logo