-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add a decimals int = 2
parameter in describe
#10188
Comments
You shouldn't round floats for formatting purposes. This is a formatting issue, the underlying data should not change. I think you should discuss on a thread regarding floating point formatting, but I think this should not be in describe. Rather in the config settings. |
@ritchie46 The issue is that a Config option would have a global impact. But I want to be able to see the actual decimal places in my main dataframe so that I know if I need to round those float columns or not. So it would make sense to control results in |
@ritchie46 |
I think it does. Formatting shouldn't be in keyword arguments. |
@ritchie46 Sure, let me rephrase that. A |
Rounding floating point values decreases the precision. |
Problem description
The mean and standard deviation results always have many decimal places.
This could be fixed with the existing Expr.round, so I don't think there will be any performance drawbacks.
We could have a default value like
decimals: int = 2
or3
so that users don't have to do expr.round themselves anymore.The text was updated successfully, but these errors were encountered: