-
Notifications
You must be signed in to change notification settings - Fork 9
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
Suggested improvement for null_lm
#17
Comments
yes, agree with this. would you make the pull request? |
I interpreted these suggestions as that you already had fixes, and could do the pull requests easily. That not being the case, then we can leave them as suggestions, and I will get to them as I can manage. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When fitting
null_lm
, you need the data to have a column.resid
otherwise it produces a graph like below. I've been manually adding the.resid
column to "true" data before, but I think it will be easier if the residual is computed also for it during thelineup
. Additionally, this method could be generalised further so that it is not just forlm
but can cater for other statistical models.Created on 2021-10-16 by the reprex package (v2.0.0)
Session info
The text was updated successfully, but these errors were encountered: