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

breaking: compatability with unfold 0.7 #8

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

breaking: compatability with unfold 0.7 #8

wants to merge 17 commits into from

Conversation

behinger
Copy link
Member

  • 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,
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
)
)

) = throw(
"The `extract_coefs` function is currently not implemented for mixed-effects models.",
)
end
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
end
end

@behinger
Copy link
Member Author

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!

@behinger
Copy link
Member Author

also closes #7

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

Successfully merging this pull request may close these issues.

1 participant