Skip to content
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

Rounded values #8

Closed
DemGrg opened this issue Dec 28, 2017 · 3 comments
Closed

Rounded values #8

DemGrg opened this issue Dec 28, 2017 · 3 comments

Comments

@DemGrg
Copy link

DemGrg commented Dec 28, 2017

Somehow now the values given by the broken function are always rounded vs exactv (tested lm model):

f.e.
contribution
(Intercept) 2300
tenure = 3 -1900
MonthlyCharges = 74.4 350
SeniorCitizen = 0 14
final_prognosis 720
baseline: 0

while: predict(model, data_in_test[analysed_user,]) gives 721.206

@pbiecek
Copy link
Owner

pbiecek commented Jan 2, 2018

This is because signif(digits=2) function was used for rounding.
But I agree that it can be misleading, in the latest version both print() and plot() functions have additional arguments digits and rounding_function so user may request a specific behaviour. And by default it's round(digits=3).

pbiecek added a commit that referenced this issue Jan 2, 2018
@pbiecek
Copy link
Owner

pbiecek commented Jan 2, 2018

Here: https://pbiecek.github.io/breakDown/articles/break_lm.html you can find some examples how to use both arguments

@DemGrg
Copy link
Author

DemGrg commented Jan 2, 2018

excellent, thank you....
works great now

@DemGrg DemGrg closed this as completed Jan 2, 2018
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

No branches or pull requests

2 participants