Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
update - I'm consulting with some teammates about what it might mean to
also support Bayesian variations on split quality, since there has been a lot
of interest in the last few years
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
I think targeting 3.0 with a refactor makes the most sense. There's no way
to do this without making small breaking changes, but slightly larger changes
could clean up the design.
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13440
Yeah I take your point that the trait Impurity already defines two methods,
only one of which is implemented for each of the subclasses. It's already a
funky design that probably should have been
Github user willb commented on the issue:
https://github.com/apache/spark/pull/13440
@srowen @thunterdb this PR passes all tests and merges cleanly -- can you
take another look? It's been open for quite a while now.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95018/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #95018 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95018/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #95018 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95018/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94042/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #94042 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94042/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #94042 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94042/testReport)**
for PR 13440 at commit
Github user jsigee87 commented on the issue:
https://github.com/apache/spark/pull/13440
Is this still being considered?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user willb commented on the issue:
https://github.com/apache/spark/pull/13440
I agree with @felixcheung -- @srowen or @thunterdb, can you take a look at
this?
---
-
To unsubscribe, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/13440
@srowen @thunterdb any more thoughts on this?
how about @sethah @yanboliang @jkbradley?
---
-
To unsubscribe, e-mail:
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@srowen I discuss some of these questions in the [blog
post](http://erikerlandson.github.io/blog/2016/05/26/measuring-decision-tree-split-quality-with-test-statistic-p-values/),
but the tl/dr
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13440
I've not seen chi squared used as a split statistic; when is it
theoretically better than entropy? Or something a bit more fundamental like KL
divergence? It makes some sense but does require some
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/13440
I don't have ML knowledge enough to review this. I can cc ML committer guys
who I can guess have some expertise from git blame but I hope there are some
sign-offs left here from some guys here
Github user willb commented on the issue:
https://github.com/apache/spark/pull/13440
@HyukjinKwon @thunterdb can you all take a look at this? It's been under
review for quite a long time!
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80093/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #80093 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80093/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #80093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80093/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
rebased to latest head of master
---
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 willb commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb can you take a look at this now that 2.2 is out?
---
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 erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
Ready for review
---
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 and wishes
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb this is ready for any further review
---
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 erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb patch is clean again. @HyukjinKwon found my problem.
---
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/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76041/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #76041 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76041/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #76041 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76041/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75944/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75944 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75944/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75944 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75944/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb still can't diagnose what the source of this "fails to generate
doc" error is. I don't see anything wrong with the scaladoc.
---
If your project is set up for it, you can reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75923/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75923 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75923/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75923/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75868/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75868/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75868/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb not sure what is failing in the CI.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75836/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75836/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75836 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75836/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75827/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75827/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75827/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75812/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75812 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75812/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75812 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75812/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
test 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 this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #75782 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75782/testReport)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75782/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #75782 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75782/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb apologize for the latency. I removed the default method defs
and rebased
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/13440
@thunterdb Can you take a look? Thanks!
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73008/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #73008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73008/testReport)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #73008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73008/testReport)**
for PR 13440 at commit
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/13440
test 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 this feature
enabled and wishes so,
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/13440
@erikerlandson I am just helping clearing the stale PRs. :) I have no idea
whether they have intention to accept it.
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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 shaneknapp commented on the issue:
https://github.com/apache/spark/pull/13440
i stopped the build as i need to restart jenkins... i'll retrigger this
when we're back up and running.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73006/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #73006 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73006/testReport)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
Hi @wangmiao1981,
I am still interested in this, but I don't have any sense about whether
upstream has any interest. Does upstream have any intention to accept it?
---
If your
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/13440
@erikerlandson Are you still working on this PR? Thanks! Miao
---
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
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@holdenk @jkbradley looks like it's clean again
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66766/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
**[Test build #66766 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66766/consoleFull)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #66766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66766/consoleFull)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
test 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 this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66756/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #66756 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66756/consoleFull)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #66756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66756/consoleFull)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66679/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #66679 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66679/consoleFull)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #66679 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66679/consoleFull)**
for PR 13440 at commit
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/13440
@holdenk yes, I'll rebase it this week.
---
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
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13440
Is this something your still working on? If so it would be good to merge in
the latest master. We can also check with @jkbradley to see if he has some
review bandwidth.
---
If your project is set
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64309/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #64309 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64309/consoleFull)**
for PR 13440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #64309 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64309/consoleFull)**
for PR 13440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13440
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/13440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59751/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13440
**[Test build #59751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59751/consoleFull)**
for PR 13440 at commit
1 - 100 of 110 matches
Mail list logo