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

Mixed 2-way ANOVA #136

Closed
dokato opened this issue Sep 30, 2020 · 4 comments · Fixed by #179
Closed

Mixed 2-way ANOVA #136

dokato opened this issue Sep 30, 2020 · 4 comments · Fixed by #179
Labels
feature request 🚧 New feature or request help wanted 🔔 Extra attention is needed

Comments

@dokato
Copy link

dokato commented Sep 30, 2020

My RM ANOVA works fine:

pg.rm_anova(data = df, dv='time', subject='pid', within=['axis', 'congr'])

I thought it would be easy to extrapolate to mixed ANOVA by introducing another between subject variable. But this:

pg.mixed_anova(data=df, dv='time', within=['axis', 'congr'], between='shift', subject='pid')

errors: 'DataFrame' object has no attribute 'dtype'

Is there a plans to introduce 2-way mixed ANOVA, eg. for a design 2x2x2?

@raphaelvallat
Copy link
Owner

Hi @dokato,

For now, only one-within / one-between factors are supported in mixed_anova. It would actually require a lot of changes to the code to add support for multiple between or within factor, so I doubt this will be implemented in the near future.

For more complex ANOVA designs, I strongly recommend using JASP.

Thanks!
Raphael

@raphaelvallat raphaelvallat added the feature request 🚧 New feature or request label Sep 30, 2020
@dokato
Copy link
Author

dokato commented Oct 1, 2020

@raphaelvallat ah pity, thanks for adding it to the roadmap though ;)

@raphaelvallat raphaelvallat added the help wanted 🔔 Extra attention is needed label Feb 13, 2021
@sappelhoff
Copy link
Contributor

based on how often this question comes up (also on gitter), it may be worthwhile to add a short check for mixed_anova: if more than one within or between factor are specified, a ValueError is raised, saying that this is not yet implemented, and pointing to this issue, asking for somebody to implement it :-)

@raphaelvallat
Copy link
Owner

Good idea @sappelhoff! Please feel free to submit it as a PR, or else I will do it for the next stable release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request 🚧 New feature or request help wanted 🔔 Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants