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

Added more tests of fit_beta_mixture and meta_analytic to reproduce known examples from R #16

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

brockk
Copy link
Collaborator

@brockk brockk commented Jan 10, 2024

Note - this branch branches from 3_fit_beta_mixture because that branch contains pre-requisite code that is not yet in main.

Copy link

codecov bot commented Jan 10, 2024

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 ☂️

@danielinteractive
Copy link
Collaborator

Thanks @brockk - do you maybe want to first merge this into my branch with this PR?
Will have a look later at contents

@brockk brockk changed the base branch from main to 3_fit_beta_mixture January 10, 2024 13:25
@brockk
Copy link
Collaborator Author

brockk commented Jan 10, 2024

Thanks @brockk - do you maybe want to first merge this into my branch with this PR? Will have a look later at contents

Yes of course, good idea. I have updated that.

Copy link
Collaborator

@danielinteractive danielinteractive left a 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

@@ -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"
Copy link
Collaborator

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?

Copy link
Collaborator Author

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 std(mix_small) ≈ std(pi_star) atol = 0.01


# Large historical dataset
Copy link
Collaborator

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)


Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

@danielinteractive danielinteractive merged commit 44fd982 into 3_fit_beta_mixture Jan 11, 2024
4 checks passed
@danielinteractive danielinteractive deleted the 2_3_more_tests branch February 22, 2024 20:14
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.

2 participants