Skip to content

Residual sum vs. regression sum #9312

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
jonvegards opened this issue Jul 10, 2017 · 1 comment · Fixed by #9314
Closed

Residual sum vs. regression sum #9312

jonvegards opened this issue Jul 10, 2017 · 1 comment · Fixed by #9314
Labels
Documentation Easy Well-defined and straightforward way to resolve

Comments

@jonvegards
Copy link

jonvegards commented Jul 10, 2017

Hi,

A short question regarding the documentation for the GradientBoostingRegressor. In the documenation for the score method to this regressor we can read that it gives the R^2 value, which is defined as 1 - u/v. Further on we read that u = ((y_true - y_pred) ** 2).sum() and is called the regression sum and that v = ((y_true - y_true.mean()) ** 2).sum() and is called the residual sum. I found this a bit strange and looked it up at Wikipedia which indeed said that u must be the residual sum and v the regression sum. I though you wanted to know this minor error.

Cheers

@jnothman jnothman added Documentation Easy Well-defined and straightforward way to resolve Need Contributor labels Jul 10, 2017
@jnothman
Copy link
Member

Thanks. A pull request is welcome if it's not already fixed in masteri

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Easy Well-defined and straightforward way to resolve
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants