openpharma / mmrm

Mixed Models for Repeated Measures (MMRM) in R.
https://openpharma.github.io/mmrm/
Other
135 stars 23 forks source link

signal non-convergence if theta_vcov is not positive definite #468

Closed danielinteractive closed 2 months ago

danielinteractive commented 2 months ago

closes #442

github-actions[bot] commented 2 months ago

Unit Tests Summary

    1 files     46 suites   27s :stopwatch:   514 tests   474 :white_check_mark: 40 :zzz: 0 :x: 1 974 runs  1 925 :white_check_mark: 49 :zzz: 0 :x:

Results for commit fa396009.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 2 months ago

Unit Test Performance Difference

Additional test case details | Test Suite | $Status$ | Time on `main` | $±Time$ | Test Case | |:-----|:----:|:----:|:----:|:-----| | tmb | 👶 | | $+0.01$ | h_mmrm_tmb_check_conv_warns_if_theta_vcov_is_not_positive_definite | | tmb | 👶 | | $+0.01$ | h_mmrm_tmb_check_conv_warns_if_theta_vcov_is_not_symmetric | | tmb | 👶 | | $+0.01$ | h_mmrm_tmb_check_conv_warns_if_theta_vcov_is_numerically_singular |

Results for commit b536320f2f903ce626863e15834a29932fb2bb4f

♻️ This comment has been updated with latest results.

github-actions[bot] commented 2 months ago

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  ----------------------------
R/between-within.R             59       0  100.00%
R/component.R                  69       0  100.00%
R/cov_struct.R                 97       1  98.97%   407
R/empirical.R                   7       0  100.00%
R/fit.R                       229       3  98.69%   420, 481, 511
R/interop-car.R               130       1  99.23%   9
R/interop-emmeans.R            51       0  100.00%
R/interop-parsnip.R            59       1  98.31%   12
R/kenwardroger.R               92       2  97.83%   41, 63
R/mmrm-methods.R              140       0  100.00%
R/residual.R                    8       0  100.00%
R/satterthwaite.R             116      12  89.66%   238-249
R/skipping.R                    8       0  100.00%
R/testing.R                    64       4  93.75%   29, 31, 80, 82
R/tidiers.R                    72       2  97.22%   46-47
R/tmb-methods.R               294       3  98.98%   162, 306-307
R/tmb.R                       305       1  99.67%   206
R/utils-formula.R              27       0  100.00%
R/utils-nse.R                  16       0  100.00%
R/utils.R                     212      12  94.34%   279-289, 449, 478
R/zzz.R                        72      26  63.89%   7-26, 59-64, 94, 122, 142
src/chol_cache.h               63       0  100.00%
src/covariance.h              101       1  99.01%   177
src/derivatives.h             126       0  100.00%
src/empirical.cpp              72       0  100.00%
src/exports.cpp                47       0  100.00%
src/jacobian.cpp               47       1  97.87%   54
src/kr_comp.cpp                56       0  100.00%
src/mmrm.cpp                   76       0  100.00%
src/predict.cpp                93       0  100.00%
src/test-chol_cache.cpp        58       5  91.38%   9, 18, 26, 55, 62
src/test-covariance.cpp       123       5  95.93%   9, 29, 40, 61, 72
src/test-derivatives.cpp      108       7  93.52%   44, 53, 62, 85, 94, 106, 124
src/test-utils.cpp            195       7  96.41%   9, 16, 24, 34, 44, 57, 119
src/testthat-helpers.h         15       5  66.67%   36-37, 41, 50, 53
src/utils.h                    78       0  100.00%
TOTAL                        3385      99  97.08%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  -------
R/tmb.R          +4       0  +0.00%
TOTAL            +4       0  +0.00%

Results for commit: fa39600999b65f166897b7b8e3f97e37a8a7e0a9

Minimum allowed coverage is 80%

:recycle: This comment has been updated with latest results