-
Notifications
You must be signed in to change notification settings - Fork 11
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
v4.4.1 #162
Conversation
…to shift_loglik
Low value
…to shift_loglik
Shift loglikelihood bugfixes
@joshwlambert @rsetienne , could you please take a quick look at the |
It would be useful to link the pkgdown website in the About section on the DAISIE github repo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've reviewed the changes and everything looks good to me.
There is nothing missing from the NEWS file that I could see other than extremely minor changes, e.g. explicit namespacing of tests.
I would suggest submitting to CRAN in order to easily use the new changes when running models locally and on the cluster.
As far as I can see the test that has the numerical discrepancy issue has now been skipped. Would it be possible to log an issue stating which test (file name and line number) is causing the issue? This can then be tackled in a follow-up PR. |
@rsetienne @Neves-P It would be great to get these changes merged and a new version of DAISIE sent to CRAN to easily install the package across different platforms. |
This looks good to me, I just added a tolerance again and unskipped the test after having checked with Hanno and @rsetienne that these numerical differences are to be expected. @rsetienne , as soon as GHA are done, could you please merge or should I go ahead (there is already one review, so I can merge it). |
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## master #162 +/- ##
==========================================
- Coverage 68.54% 67.11% -1.43%
==========================================
Files 78 79 +1
Lines 7964 8123 +159
==========================================
- Hits 5459 5452 -7
- Misses 2505 2671 +166
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
DAISIE 4.4.1
Inf
Relaxed-rate model cannot take Inf K as initial parameter #159 Error when relaxed parameter mean or sd is infinite #160 (@joshwlambert).DAISIE_SR_loglik_CS_M1()
is nowDAISIE_loglik_CS_shift()
.