[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-24 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14808 @sethah I seem to have missed the other comment with the notes what has to be done to make the PR an example for the change. Sorry. Since I'm very new to it and the only way to learn

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-23 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 I think we should close this. The new example and the user guide should be updated against [SPARK-17239](https://issues.apache.org/jira/browse/SPARK-17239). @jaceklaskowski If you'd still like to

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-20 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 I think we should add an example for multiclass classification, to demonstrate that it works in 2.1+. We also need to update the documentation that says we only support binary. --- If your

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-20 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14808 @sethah Anything to improve the example. I'm open for suggestions and improve the example (to learn that part better). Thanks! --- If your project is set up for it, you can reply to this

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-19 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 https://github.com/apache/spark/pull/14834 is merged now. We did not implement a new API, but we can still update the logistic regression examples to show the new multiclass functionality. --- If

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-12 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14808 Thanks @MLnick. What would you suggest with the PR then (after the comments from @sethah)? Please guide. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-12 Thread MLnick
Github user MLnick commented on the issue: https://github.com/apache/spark/pull/14808 by the way multi-class LR will be in 2.1 not 2.0.1, so we do have some time to wait for #14834 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-01 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 @jaceklaskowski You can see https://github.com/apache/spark/pull/14834 --- 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] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-09-01 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14808 Any news on that @sethah? I've seen some discussions about the changes to unify the interfaces, but am wondering how close the other PRs are so I could help with that one myself. --- If

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-26 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 We are trying to move quite fast on 17163. I will submit a PR for it soon (not sure how long review will take). The issue here is that the current example will show users how to use an estimator

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-26 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/14808 I don't mind adapting the example to the upcoming changes and I can work on it. But.. SPARK-17163 has no Fix Version/s field assigned so I'm reading it that it's not clear whether

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-25 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/14808 This is going to have to be changed after [SPARK-17163](https://issues.apache.org/jira/browse/SPARK-17163). Sorry about the confusion! We'll still want to make an example with multiclass, though, so

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14808 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] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14808 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64423/ Test PASSed. ---

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14808 **[Test build #64423 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64423/consoleFull)** for PR 14808 at commit

[GitHub] spark issue #14808: [SPARK-17156][ML][EXAMPLE] Add multiclass logistic regre...

2016-08-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14808 **[Test build #64423 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64423/consoleFull)** for PR 14808 at commit