-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
correlation formatting doesn't apply digits
to the CI formatting
#337
Comments
correlation::correlation(iris, digits=1, ci_digits = 5)
#> # Correlation Matrix (pearson-method)
#>
#> Parameter1 | Parameter2 | r | 95% CI | t(148) | p
#> ------------------------------------------------------------------------------
#> Sepal.Length | Sepal.Width | -0.1 | [-0.27269, 0.04351] | -1.4 | 0.152
#> Sepal.Length | Petal.Length | 0.9 | [ 0.82704, 0.90551] | 21.6 | < .001***
#> Sepal.Length | Petal.Width | 0.8 | [ 0.75690, 0.86484] | 17.3 | < .001***
#> Sepal.Width | Petal.Length | -0.4 | [-0.55088, -0.28795] | -5.8 | < .001***
#> Sepal.Width | Petal.Width | -0.4 | [-0.49721, -0.21870] | -4.8 | < .001***
#> Petal.Length | Petal.Width | 1.0 | [ 0.94905, 0.97299] | 43.4 | < .001***
#>
#> p-value adjustment method: Holm (1979)
#> Observations: 150 Created on 2025-01-19 with reprex v2.1.1 |
what about the inconsistency with |
I would still have by default |
Ok, we just need to add it to the print and/or format methods, I guess? |
I think so probably here correlation/R/methods_format.R Lines 4 to 9 in 94f3d0a
|
Received by email:
CI stays with 2 digits (but it works in
parameters
)Created on 2025-01-19 with reprex v2.1.1
The text was updated successfully, but these errors were encountered: