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

Run power posterior analyses stone-by-stone #397

Merged
merged 3 commits into from
Oct 23, 2023

Conversation

davidcerny
Copy link
Contributor

This branch provides a Rev interface to the runStone() and summarizeStones() methods of the class PowerPosteriorAnalysis, making it possible to analyze individual beta values / power posteriors (or "stones") one by one instead of all at once. This should come in handy for running computationally intensive marginal likelihood analyses on machines with a limited amount of memory and/or number of cores, since it means that analyses of a large number of stones can now be executed in batches instead of all at once.

I have also updated the corresponding test suite, which now demonstrates that the run-all vs. run-stones-one-at-a-time implementations produce exactly the same marginal likelihoods when using the single-thread version of RevBayes. For the MPI version, the run-stones-one-at-a-time implementation still produces exactly the same marginal likelihood as with the single-thread version, but the run-all implementation deviates from its single-thread counterpart. This deviation is not new; it already existed in the previous version of the suite, and was actually much more drastic (reaching ~5,500 log units instead of the present ~25).

@hoehna hoehna merged commit 90f4e85 into development Oct 23, 2023
19 checks passed
@hoehna hoehna deleted the single-stone-power-posterior branch October 23, 2023 13:01
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.

None yet

2 participants