Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I have been working on moving the Gadfly plotting functionality from my package into MCMCChains.jl. However, I figured that I was basically
Gadfly.plot(chn::Chains, args...; vargs..., filter=nothing)
chn
todf::DataFrame
filter
todf
Gadfly.plot(df, args...; vargs...)
So, most of this could be replaced by using the DataFrames constructor provided by MCMCChains.jl. Since this is easy in use, but can be hard to figure out, I've added a few examples to the docs.
Some examples from the generated documentation:
I wonder if parts of these examples should be implemented in
src
. I don't think so given the relative easy of use, but am not sure about it.What would possibly be interesting is to implement a function which uses
Gadfly.hstack
to show the density and trace side by side. From my package:For the examples, creating such a hstack was a bit too involved.
Vertical bars for credibility intervals
On a side-note, my package was also showing vertical bars for credibility intervals, like
This functionality is hopefully going to be merged into Gadfly, see GiovineItalia/Gadfly.jl#1521.