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 unit tests from README #15

Merged
merged 14 commits into from
Jun 21, 2024
Merged

Added unit tests from README #15

merged 14 commits into from
Jun 21, 2024

Conversation

wleoncio
Copy link
Member

This is work to close issue #13.

@wleoncio wleoncio added the enhancement New feature or request label Jun 21, 2024
@wleoncio wleoncio requested a review from zhizuio June 21, 2024 09:05
@wleoncio
Copy link
Member Author

macOS check failed due not minor ($10^{-2}$) differences in values of pred_2$cumhazard and pred_2$survival. I'm assuming this is not relevant, so I'll relax the test expectations a bit.

So checks passes on macOS
@zhizuio zhizuio merged commit 3e5e1a1 into main Jun 21, 2024
4 checks passed
@zhizuio
Copy link
Collaborator

zhizuio commented Jun 21, 2024

The minor differences of the results are expected because of the short MCMC in test-general.R.

@wleoncio wleoncio deleted the issue-13 branch June 21, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating an example of BayesSurvive(MRF.G = FALSE)
2 participants