-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
refactoring the correlation
package
#298
Comments
Hi Tom, welcome on board! Great to see you participating in the easystats project! 🎉 |
Welcome Tom! :) good luck with all this work 😉 |
Hi @TomGeva good to have you |
BTW, I have some code and a precompiled model (written for a client) to estimate full Bayesian correlation matrices (and partial correlation matrices) using |
Happy to join!, thanks for the kind words 😁 |
Yeah it'd be great to have more flexibility in terms of prior setting and all, assuming it preserves some of the advantages of BayesFactor (you don't need to compile stan stuff, it's fast and it gives BFs out of the box) |
As discussed in out meeting yesterday, the correlation package should be broken down (and built up!) into the following "bits":
A simple 1:1 correlation function (currently done in
cor_test()
(#261, #260)
BayesFactor
)The methods for tetrachoric, polychoric, and biserial correlations can be improved, I think.
Things left to do:
cor_test()
TODO
sx
andy
to be vectors?A correlation "matrix" function
(#292, #217, #232)
select
arguments.summary()
method).A function for part/partial correlation
(#311, #301, #204?, #181)
This function will also produce multilevel correlations (#253, #207)?
select
arguments to controlsummary()
method).Things to keep
see
are good.cor_*()
functions also, I think?is.cor
andisSquare
(maybe rename to snake-case?)z_fisher()
(rename tofishers_z
, which is more inline with "named" statistic convention ineffectsize
?)Also welcoming @TomGeva that will be working on this with @bwiernik and myself
WIP can be found here: https://github.com/TomGeva/correlation2
The text was updated successfully, but these errors were encountered: