-
Notifications
You must be signed in to change notification settings - Fork 17
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
Quickly plot marginals for parameter distribution #264
Conversation
54e53c8
to
8ab7339
Compare
NB: codecov doesn't like this because it only tests the PlotRecipes if plots are enabled (and github actions doesn't enable them) |
82273c5
to
9bccc03
Compare
bors try |
tryBuild failed: |
Note that bors has succeeded on a rerun of the test suite, it seems like an unrelated test of inflation has a slightly too-tight tolerance which is subject to failure on random-seed change - see here for the refresh: |
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.
The changes look sensible to me! The only thing left is to let Bors do its thing
added plots example into docs format add RecipesBase actually add file... formatting... rm "using plots" format... normalize histograms to PDF
6d99089
to
eb5efb6
Compare
bors r+ |
Build succeeded: |
Purpose
Closes #254
Content
plot(pd::ParameterDistribution)
plot(pd <: ParameterDistributionType)
ENV["CES_TEST_PLOT_OUTPUT"] = true
prior to] test
)In future
I expect these can be improved upon in various ways, using other plot-types, or only plotting unique distributions where they are repeated etc. but for this PR I just wanted something simple to work on any case.
Example from
test/
A complicated distribution:
plot(u)