JWiley / brmsmargins

Average marginal estimates for Bayesian models fit with the brms package
https://joshuawiley.com/brmsmargins
Other
20 stars 2 forks source link

Fix Documentation & Helpful Errors #3

Closed JWiley closed 2 years ago

JWiley commented 2 years ago

Thanks to feedback / suggestions from @jwilliamrozelle several priority areas to improve are:

jwrozelle commented 2 years ago

Hope all is well there! Since this was so minor, I just added the error message in myself and submitted a pull request.

Also happy to contribute tests, errors, or anything else on the R programming side if you'd find it helpful so that you can focus on other things in the package. I'm not proficient in C++, and wouldn't want to tinker with any of the actual calculations, but, know my way around package development essentials.

JWiley commented 2 years ago

@jwilliamrozelle Thanks, I appreciate that. I've also pushed some more changes today with some further messages around arguments to hopefully make issues easier.

I'd definitely welcome any contributions around tests, errors, docs, or any other package improvements and would of course recognize contributions/authorship if you did. I have three bigger tasks in mind for the package, in no particular order:

I don't think it would have to be huge work but something else I wonder about is more "convenience" functions. Nice output, graphs, etc. I've not thought too much about the output format so far and what people do with the output once they have it so believe there is probably room for improvement there & would definitely be on the R side of things.

JWiley commented 2 years ago

Done now in v0.2.0

jwrozelle commented 2 years ago

Whoops, looks like I missed this. Happy to touch base at some point if I can help on any of the above.

I should have a bit more flexibility this summer to support, if I can be helpful.

JWiley commented 2 years ago

@jwilliamrozelle No problem. Over summer if you've got some time, please reach out & would love any contributions.