-
Notifications
You must be signed in to change notification settings - Fork 304
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
Tidy output for lmer is not very useful #96
Comments
The change in 4.0 looks like an API change that should be documented in the release notes because it will break old code. Here are my notes on what's happened... Old behavior:
Where it was actually calling The new default is random effects variances and fixed effects estimates:
To replicate the old default, we have to call
So the release notes should say something like:
|
I agree that this is an API break (it's my fault). Some comments:
Hadley, can you give a use case? Your request seems like more the output of |
tidy(mymodel, "ran_modes") doesn't work.
|
@skanskan, I don't doubt this, but do you happen to have a reproducible example? Works in my fork, but I'm not 100% sure that it's not divergent from the current master branch in some way ... For me,
works fine. (Are you using the CRAN version or the latest Github version?) |
I'm going to continue this conversation here, I think there are few things that need pointing out:
|
Bump. Any further comments? |
It would be great if tidy(model, "fixed") also included the column "Pr(>|t|)" added by lmerTest. |
bump. I think I'm talking to myself a bit here: I would really like to move this forward a bit, and I have moderately strong opinions about what the "right way" is (i.e. the default is to return the fixed-effect coefficients and the variance-covariance parameters, appropriately identified; the old behavior, which returns the output of |
bump ... ? |
Hi Ben: thank you so much for your interest, involvement, and commitment. The truth is that I know so little about linear mixed effects models that I will not be of any help here, so I'm very open to trusting your opinions. Since you're the maintainer of lme4, have you considered implementing the tidy/augment/glance methods in lme4, after importing and re-exporting the generics from broom? This is a pattern I've recently started encouraging among other maintainers who are familiar with broom, and it would remove me as an obstacle. If you would rather this live in broom, please do submit this as a pull request and I promise to be more punctual about taking a look and merging it in. |
Hmm. Thanks for the ideas. (I was hoping to get feedback from someone else who uses mixed models - maybe I'll advertise this issue at |
@bbolker @skanskan It seems
|
@vathymut, I've fixed this problem in my fork. However, see my comment above: You may also be interested to know that the development version of lme4 now includes an |
@bbolker Thank you. |
I think this should be closed. @hadley, I'm particularly interested in your opinions, as the originator of the issue ... |
Closing in favor of bbolker/broom.mixed#7. |
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue. |
I'd rather see something like
coef(fm1)
- i.e. the individual estimates of the random effects for each groupThe text was updated successfully, but these errors were encountered: