Skip to content

Conversation

@yanboliang
Copy link
Contributor

What changes were proposed in this pull request?

Provide R-like summary statistics for GLMs via iteratively reweighted least squares.

How was this patch tested?

unit tests.

@SparkQA
Copy link

SparkQA commented Mar 14, 2016

Test build #53066 has finished for PR 11694 at commit 5d4c87b.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Mar 14, 2016

Test build #53067 has finished for PR 11694 at commit fba1112.

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


new IterativelyReweightedLeastSquaresModel(model.coefficients, model.intercept)
new IterativelyReweightedLeastSquaresModel(model.coefficients,
model.intercept, model.diagInvAtWA, iter)
Copy link
Contributor

Choose a reason for hiding this comment

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

new IterativelyReweightedLeastSquaresModel(
  model.coefficients, model.intercept, model.diagInvAtWA, iter)

@mengxr
Copy link
Contributor

mengxr commented Mar 14, 2016

I made one pass and left some minor comments line. This looks great overall!

@SparkQA
Copy link

SparkQA commented Mar 16, 2016

Test build #53269 has finished for PR 11694 at commit f89cdf0.

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

@mengxr
Copy link
Contributor

mengxr commented Mar 16, 2016

LGTM. Merged into master. Thanks!

roygao94 pushed a commit to roygao94/spark that referenced this pull request Mar 22, 2016
…eweighted least squares

## What changes were proposed in this pull request?
Provide R-like summary statistics for GLMs via iteratively reweighted least squares.
## How was this patch tested?
unit tests.

Author: Yanbo Liang <[email protected]>

Closes apache#11694 from yanboliang/spark-9837.
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.

3 participants