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

BACKPORT: LRT for comparing non-mixed to mixed models #510

Merged
merged 3 commits into from
Apr 15, 2021
Merged

Conversation

palday
Copy link
Member

@palday palday commented Apr 15, 2021

cf #508.

palday and others added 3 commits April 15, 2021 20:10
* LRT  for comparing non-mixed to mixed models

* nesting for LM/GLMM and GLM/LMM

* NEWS entry

* use _iscomparable instead of isnested

* add missing dep

* inner has no more preds than outer, thx @kleinschmidt

* column span check

* oops

* julia 1.4 compat

* condition LRT show method on G/LMM

* use deviance for GLM and -2 loglikelihood for LMM

* contains -> occursin (Julia 1.4 compat)

* Apply suggestions from code review

Co-authored-by: Dave Kleinschmidt <[email protected]>

* copy-paste error

* reenable a shortcut

* Apply suggestions from code review

Co-authored-by: Douglas Bates <[email protected]>

Co-authored-by: Dave Kleinschmidt <[email protected]>
Co-authored-by: Douglas Bates <[email protected]>
(cherry picked from commit 365e5f7)
@codecov
Copy link

codecov bot commented Apr 15, 2021

Codecov Report

Merging #510 (43948a5) into v3.x (0cc0d29) will increase coverage by 0.14%.
The diff coverage is 100.00%.

❗ Current head 43948a5 differs from pull request most recent head 93b6e36. Consider uploading reports for the commit 93b6e36 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             v3.x     #510      +/-   ##
==========================================
+ Coverage   93.91%   94.06%   +0.14%     
==========================================
  Files          24       24              
  Lines        2105     2155      +50     
==========================================
+ Hits         1977     2027      +50     
  Misses        128      128              
Impacted Files Coverage Δ
src/generalizedlinearmixedmodel.jl 88.80% <100.00%> (+0.08%) ⬆️
src/likelihoodratiotest.jl 98.49% <100.00%> (+0.84%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0cc0d29...93b6e36. Read the comment docs.

@palday palday merged commit d5db680 into v3.x Apr 15, 2021
@palday palday deleted the pa/unmixedlrt3 branch April 15, 2021 18:53
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