-
Notifications
You must be signed in to change notification settings - Fork 0
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
Added more tests of fit_beta_mixture and meta_analytic to reproduce known examples from R #16
Conversation
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
Thanks @brockk - do you maybe want to first merge this into my branch with this PR? |
Yes of course, good idea. I have updated that. |
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.
looks great, just few minor comments
design/ReconcileWithR.jl
Outdated
@@ -335,7 +335,7 @@ Turing = "~0.29.3" | |||
PLUTO_MANIFEST_TOML_CONTENTS = """ | |||
# This file is machine-generated - editing it directly is not advised | |||
|
|||
julia_version = "1.9.3" | |||
julia_version = "1.9.1" |
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.
are you sure you want to use a lower version? maybe even upgrade to 1.10 could be interesting?
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.
This was of course a change automatically generated by Pluto that reflects my Julia version is slightly older than the previous committer (I am guessing you, cos I have not changed my Julia install). I am just going to roll this change back. I like the idea of trying under 1.10 but not today!
test/test_fit_beta_mixture.jl
Outdated
@test std(mix_small) ≈ std(pi_star) atol = 0.01 | ||
|
||
|
||
# Large historical dataset |
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.
maybe split this in 2 tests, one for small, one for large dataset?
check_numerical(map_small, [:a], [0.17], atol = 0.01) | ||
check_numerical(map_small, [:b], [0.51], atol = 0.01) | ||
|
||
|
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.
same here
Note - this branch branches from 3_fit_beta_mixture because that branch contains pre-requisite code that is not yet in main.