-
Notifications
You must be signed in to change notification settings - Fork 4
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
Individual Loo for submodels #408
Conversation
Code Coverage Summary
Diff against main
Results for commit: bb458ad Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 178 suites 6m 9s ⏱️ Results for commit bb458ad. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit 6827661 ♻️ This comment has been updated with latest results. |
Actually before merging I think it might be worth adding a few CRON tests that just compare the loo values to that of brms to ensure consistency. |
@gravesti - addressed your comment :) - Also added an |
@gowerc I think you might not have pushed your latest changes yet :) |
I think I might have just added that comment on the wrong PR, looking over it I've confused myself now. But the "URL" change looks like it was introduced in the Gamma model PR so maybe I made the change on the wrong branch... |
@gravesti - Looks like merging in main dismissed your approval sorry. |
Closes #402