-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documentation of alpha #831
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noting the prior change was intentional based on prior predictives but happy for it to be widened. It might be good to split this out from the doc change.
Agree with splitting out the prior change. |
This reverts commit 68c7b03.
@seabbs just fyi I think the failure here is again related to some issue with the caching in the cmdstan action on macOS. Will clear and re-trigger checks. |
I also cleared it an reran. I think a patch fix here would be to set the cache to auto-delete after say a week . Might strike a better balance |
Description
This PR closes #829.
It clarifies the documentation of alpha.
Initial submission checklist
devtools::test()
anddevtools::check()
).devtools::document()
).lintr::lint_package()
).After the initial Pull Request