Skip to content

[SPARK-6252] [mllib] Added getLambda to Scala NaiveBayes #4969

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

[SPARK-6252] [mllib] Added getLambda to Scala NaiveBayes #4969

wants to merge 2 commits into from

Conversation

@jkbradley
Copy link
Member

@jkbradley jkbradley commented Mar 10, 2015

Note: not relevant for Python API since it only has a static train method

@SparkQA
Copy link

SparkQA commented Mar 10, 2015

Test build #28445 has started for PR 4969 at commit 63eff48.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 10, 2015

Test build #28445 has finished for PR 4969 at commit 63eff48.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 10, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28445/
Test PASSed.

mengxr
mengxr reviewed Mar 12, 2015
View reviewed changes
mllib/src/test/scala/org/apache/spark/mllib/classification/NaiveBayesSuite.scala Outdated
Copy link
Contributor

@mengxr mengxr Mar 12, 2015

Choose a reason for hiding this comment

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

use === instead of ==

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28557 has started for PR 4969 at commit a471d90.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Mar 13, 2015

LGTM of course

@SparkQA
Copy link

SparkQA commented Mar 13, 2015

Test build #28557 has finished for PR 4969 at commit a471d90.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 13, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28557/
Test PASSed.

@mengxr
Copy link
Contributor

mengxr commented Mar 13, 2015

Merged into master. Thanks!

@asfgit asfgit closed this in dc4abd4 Mar 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jkbradley @SparkQA @AmplabJenkins @srowen @mengxr