mjskay / tidybayes

Bayesian analysis + tidy data + geoms (R package)
http://mjskay.github.io/tidybayes
GNU General Public License v3.0
726 stars 59 forks source link

Standardize function arguments where possible #70

Open mjskay opened 7 years ago

mjskay commented 7 years ago

Some packages use different names for the same argument (e.g. "nsamples" or "draws" for number of samples to draw for the prediction / fit). Should make a complete catalog of these and standardize them

List so far (checked once normalized):

tmastny commented 6 years ago

I think auxpars needs to be added to this list. add_predicted_samples.R technically swaps a auxpars call for dpar. In fact, this could lead to the same issue as in https://github.com/mjskay/tidybayes/pull/96, where if the user specified dpars in ... we'd get the matched by multiple actual arguments error.

I'm working on this fix with the pull request. Having this list is very helpful.

mjskay commented 6 years ago

Good point, added it to the list

mjskay commented 6 years ago

What remains here is post-CRAN (mostly MCMCglmm stuff, which I am punting to post-CRAN), so I am removing the CRAN milestone from this for now