Skip to content

[SPARK-7547][ML] Scala Example code for ElasticNet #6576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from

Conversation

dbtsai
Copy link
Member

@dbtsai dbtsai commented Jun 2, 2015

This is scala example code for both linear and logistic regression. Python and Java versions are to be added.

@SparkQA
Copy link

SparkQA commented Jun 2, 2015

Test build #33962 has finished for PR 6576 at commit 9462f5f.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(
    • case class Params(

@SparkQA
Copy link

SparkQA commented Jun 2, 2015

Test build #33966 has finished for PR 6576 at commit 1ec29d4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(
    • case class Params(

s"L1 and L2, default: ${defaultParams.elasticNetParam}")
.action((x, c) => c.copy(elasticNetParam = x))
opt[Int]("maxIter")
.text(s"maximal number of iterations, default: ${defaultParams.maxIter}")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say "maximum" since "maximal" often implies there are multiple maxima.

@jkbradley
Copy link
Member

Looks good other than those small comments. Thanks!

@SparkQA
Copy link

SparkQA commented Jun 2, 2015

Test build #34015 has finished for PR 6576 at commit 136e0dd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(
    • case class Params(

.setElasticNetParam(params.elasticNetParam)
.setMaxIter(params.maxIter)
.setTol(params.tol)
.setMaxIter(params.maxIter)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicated setMaxIter ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Fixed.

@SparkQA
Copy link

SparkQA commented Jun 2, 2015

Test build #34027 has finished for PR 6576 at commit 6bb6d77.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(
    • case class Params(

@SparkQA
Copy link

SparkQA commented Jun 3, 2015

Test build #34033 has finished for PR 6576 at commit e7ca406.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(
    • case class Params(

@jkbradley
Copy link
Member

LGTM, merging with master and branch-1.4
Thanks!

asfgit pushed a commit that referenced this pull request Jun 3, 2015
This is scala example code for both linear and logistic regression. Python and Java versions are to be added.

Author: DB Tsai <dbt@netflix.com>

Closes #6576 from dbtsai/elasticNetExample and squashes the following commits:

e7ca406 [DB Tsai] fix test
6bb6d77 [DB Tsai] fix suite and remove duplicated setMaxIter
136e0dd [DB Tsai] address feedback
1ec29d4 [DB Tsai] fix style
9462f5f [DB Tsai] add example

(cherry picked from commit a86b3e9)
Signed-off-by: Joseph K. Bradley <joseph@databricks.com>
@asfgit asfgit closed this in a86b3e9 Jun 3, 2015
@dbtsai dbtsai deleted the elasticNetExample branch June 9, 2015 04:27
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
This is scala example code for both linear and logistic regression. Python and Java versions are to be added.

Author: DB Tsai <dbt@netflix.com>

Closes apache#6576 from dbtsai/elasticNetExample and squashes the following commits:

e7ca406 [DB Tsai] fix test
6bb6d77 [DB Tsai] fix suite and remove duplicated setMaxIter
136e0dd [DB Tsai] address feedback
1ec29d4 [DB Tsai] fix style
9462f5f [DB Tsai] add example
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
This is scala example code for both linear and logistic regression. Python and Java versions are to be added.

Author: DB Tsai <dbt@netflix.com>

Closes apache#6576 from dbtsai/elasticNetExample and squashes the following commits:

e7ca406 [DB Tsai] fix test
6bb6d77 [DB Tsai] fix suite and remove duplicated setMaxIter
136e0dd [DB Tsai] address feedback
1ec29d4 [DB Tsai] fix style
9462f5f [DB Tsai] add example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants