combining ppc_error_scatter_avg_vs_x with ppc_error_scatter_avg(x = x) #367
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 #361, we discussed the possibility of rearranging plots for easier usage. As I mentioned in the sub-issue #364, one of the ideas I had was combining
ppc_error_scatter_avg_vs_x()
withppc_error_scatter_avg(x = x)
. The simple idea behind this is that these two functions are fairly similar, and allppc_error_scatter_avg_vs_x()
does is format thex
variable appropriately and then callppc_error_scatter_avg()
. Therefore, I think combining these two functions would result in a smoother user experience by reducing the number of similar functions.This PR addresses my work on that idea.
x
argumentppc_error_scatter_avg_vs_x()
Results
To show the results of the work, I re-created the examples used at PPC-errors documentation.
It seems like functions are working as intended, producing correct plots, and
ppc_error_scatter_avg_vs_x()
is producing the correct deprecation warning.