add x as argument to ppc_error_binned #359
Open
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.
In #358, one of the two tasks that I mentioned was adding
x
as an optional argument toppc_error_binned
so that it supports covariates in the x-axis. Asx
argument would be optional, it wouldn't be breaking any existing graphs.This PR is my work on the first task of #358.
My current implementation passes existing tests without any issue, so I think I am not breaking anything.
Examples
Here are a few examples of it working.
Default version
This default version simply follows the example presented at the PPC errors documentation.
Herd
This is the plot of average errors against herd variable.
Size
This is the plot of average errors against size variable.