Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3583
---
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, or if the feature is enab
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-96780105
@dikejiang This work is now being done here:
[https://issues.apache.org/jira/browse/SPARK-3727]
Can you please close this PR?
If you still want to work on
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-96770011
Can one of the admins verify this patch?
---
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 pro
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-77604105
@dikejiang Do you still plan to update this PR to return a Vector of
probabilities? I'm planning a major reorganization of trees & ensembles APIs
here: [https://issue
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-67049092
@dikejiang Apologies--I think I was not clear. I was recommending that
you change this PR to implement predictRaw(), rather than predictWithWeight().
Does that sound
Github user dikejiang commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-66735373
@mengxr OK to go?
---
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 fea
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-66563683
@dikejiang Great, 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
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3583#issuecomment-66348700
@dikejiang Thanks for the PR! I'm wondering if you'd be interested in a
more general API. In the new experimental ML package, I have a PR
[https://www.github.com/apac