Skip to content

[SPARK-2479 (partial)][MLLIB] fix binary metrics unit tests#1576

Closed
mengxr wants to merge 1 commit into
apache:masterfrom
mengxr:fix-binary-metrics-unit-tests
Closed

[SPARK-2479 (partial)][MLLIB] fix binary metrics unit tests#1576
mengxr wants to merge 1 commit into
apache:masterfrom
mengxr:fix-binary-metrics-unit-tests

Conversation

@mengxr

@mengxr mengxr commented Jul 24, 2014

Copy link
Copy Markdown
Contributor

Allow small errors in comparison.

@dbtsai , this unit test blocks #1562 . I may need to merge this one first. We can change it to use the tools in #1425 after that PR gets merged.

@SparkQA

SparkQA commented Jul 24, 2014

Copy link
Copy Markdown

QA tests have started for PR 1576. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17124/consoleFull

@dbtsai

dbtsai commented Jul 24, 2014

Copy link
Copy Markdown
Member

@mengxr Feel free to merge this one first. After you merge, I'll rebase #1425 against current master, and address the conflicts.

@SparkQA

SparkQA commented Jul 24, 2014

Copy link
Copy Markdown

QA results for PR 1576:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17124/consoleFull

@rxin

rxin commented Jul 24, 2014

Copy link
Copy Markdown
Contributor

Merging in master.

@asfgit asfgit closed this in c960b50 Jul 24, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Allow small errors in comparison.

@dbtsai , this unit test blocks apache#1562 . I may need to merge this one first. We can change it to use the tools in apache#1425 after that PR gets merged.

Author: Xiangrui Meng <meng@databricks.com>

Closes apache#1576 from mengxr/fix-binary-metrics-unit-tests and squashes the following commits:

5076a7f [Xiangrui Meng] fix binary metrics unit tests
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