Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #68103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68103/consoleFull)**
for PR 15314 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah Since issue of `labelCol` has been resoloved, I continue to work on
this.
Like casting of `labelCol`, I make the casting of `weightCol` happens in
`Predictor.fit()`. What's your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67974/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67974 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67974/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67967/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67967/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67974 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67974/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67965/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67965 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67965/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67967/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67965 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67965/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67956/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67956 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67956/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67956 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67956/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67955 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67955/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67955/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #67955 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67955/consoleFull)**
for PR 15314 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15314
Yeah, the original fix was small and so it was ok to include it here, but
we are expanding the scope of the change so I think it's best to separate it.
---
If your project is set up for it, you
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@srowen Feel free. It was a small fix that only modify `getNumClasses` in
that jira before we tend to move cast to `Predictor`.
---
If your project is set up for it, you can reply to this
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah that's my fault, as I originally told @zhengruifeng that I
preferred to make these changes together. They sound closely related, but, if
you all think it should be separate after looking
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah OK, I will open a new JIRA about labelCol.
---
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
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15314
Maybe we can solve the label column issue first? Would you mind opening a
new Jira/PR? I'm happy to hear other opinions as well :)
---
If your project is set up for it, you can reply to this email
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah ok, I will revert this PR to only focus on: 1, add test for
WeightCol in MLTestingUtils.checkNumericTypes; 2, add cast for WeightCol in
each algo; 3, add cast in `getNumClasses` to
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15314
I strongly prefer to move the issue with the label column into its own
Jira/PR. They are different changes and I think the label column issues are
large enough to warrant their own considerations.
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah @BenFradet I have moved the cast of `LabelCol` and `WeightCol` (if
any) into `Predict.fit`. And now algos only need to deal with casted dataframe
in its `train()` method.
---
If your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66600/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66600 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66600/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66600 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66600/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66598/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66598 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66598/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66598 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66598/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66595/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66595/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66595 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66595/consoleFull)**
for PR 15314 at commit
Github user BenFradet commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah Moving the cast to `Predictor.fit` seems like a good idea. It'd
remove the reliance on `extractLabeledPoints` which is not used by every
algorithm (eg LR).
---
If your project is set up
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66566/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66566 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66566/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66566 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66566/consoleFull)**
for PR 15314 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah @srowen I think we can firstly work on this PR to support weightCol
numeric types and by the way fix the bug in LabelCol.
And then, open another jira to discuss whether cast
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15314
The problem with the classifier tests failing reveals a more fundamental
issue. We tackle the numeric label problem by having each algorithm convert the
label column to `DoubleType`. That means that
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66441/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66441/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66441/consoleFull)**
for PR 15314 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
After discussion with @srowen , I merge the fix of 'getNumClasses' into
this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah DecisionTreeClassifierSuite, NaiveBayesSuite,
RandomForestClassifierSuite fail due to `Classifier.getNumClasses()` without
meta do not support non-double numeric label.
I think
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66421/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66421 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66421/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66420/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66420/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66421 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66421/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66420/consoleFull)**
for PR 15314 at commit
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah I have add weighCol datatype checking into `checkNumericTypes`.
---
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 issue:
https://github.com/apache/spark/pull/15314
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66364/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66364/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66364/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66355/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66355/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66355/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66320/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66320 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66320/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66319/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66319/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66320 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66320/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66319/consoleFull)**
for PR 15314 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66317/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15314
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66317/consoleFull)**
for PR 15314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15314
**[Test build #66317 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66317/consoleFull)**
for PR 15314 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/15314
@zhengruifeng We already test the label col using `checkNumericTypes` in
`MLTestingUtils`. Temporary tests are ok with me but they still need to test
every single numeric type, in each test suite.
85 matches
Mail list logo