-
Notifications
You must be signed in to change notification settings - Fork 0
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
breaking: compatability with unfold 0.7 #8
base: main
Are you sure you want to change the base?
Conversation
behinger
commented
Jun 23, 2024
- inital lmm_perm
- Moved UnfoldMixedModels also to new extension
- well, it does something now
- added lmm clusterperm
- forgot to save this file
- forgot to save
- Update UnfoldStatsMixedModelsPermutationsExt.jl
- added an 'abs'
- upgrade to unfold07
- remove compatability with unfold 0.6
coefficient; | ||
n_permutations = 10, | ||
clusterforming_threshold = 1.8, | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[JuliaFormatter] reported by reviewdog 🐶
) | |
) |
) = throw( | ||
"The `extract_coefs` function is currently not implemented for mixed-effects models.", | ||
) | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[JuliaFormatter] reported by reviewdog 🐶
end | |
end |
failling as we need a new version of Unfold released. Also don't forget to update the project.toml compat to unfold 0.7.4! |
also closes #7 |