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

Prevent return inferencedata error + cheater/liar fix #536

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

toppare
Copy link
Contributor

@toppare toppare commented Nov 7, 2021

The default of return_inferencedata was False but in the future (v4.0) it will be True. However, the rest of the code is written for False case so I changed all pm.sample to have an explicit False.

Currently it shows this warning:

FutureWarning: In v4.0, pm.sample will return an arviz.InferenceData object instead of a MultiTrace by default. You can pass return_inferencedata=True or return_inferencedata=False to be safe and silence this warning

-> Also a small change in the text in Ch2 as we are actually modeling the cheaters and not liars.

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