Skip to content
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

Error in the peer-bias example, probably due to search/replace, fixed #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Accio
Copy link

@Accio Accio commented Jan 27, 2022

Dear @rmcelreath, I went through the examples shown in the Science Before Statistics seminar, and found a small mistake in the peer-bias example. They are apparently due to an obsolete variable named I, which was later changed probably to X. In the current code, the use of variable I leads to the error that closure cannot be converted to an integer, becaue I is interpreted as a function.

I fixed the issue in this PR. In addition, I used cmdstan for the last Bayesian example because rstan reported error.

Please let me know in case you have questions or concerns. I want to thank you for all the great lectures and talks that you share. Keep moving on!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant