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

Support multichain ppc in plot_ppc #526

Merged
merged 13 commits into from
Jan 15, 2019
Merged

Conversation

ahartikainen
Copy link
Contributor

Adds support for multichain ppc.

Also implements jitter for scatter and changes alpha defaults for scatter.

@ahartikainen
Copy link
Contributor Author

I will wait from_dict before I implement tests

@ahartikainen
Copy link
Contributor Author

Now the scatter is more flexible

Data count 30 & 10

image

Data count 30 & 1000

image

@aloctavodia
Copy link
Contributor

I think the default value of jitter should be 0/None. Adding jitter can make the samples to overlap, and thus hiding the inter-sample variability.

@ahartikainen
Copy link
Contributor Author

Sounds reasonable. I will change it.

@ahartikainen
Copy link
Contributor Author

There is something wrong with travis. Forestplot works (I removed problematic code). So I think this can wait until it is merged.

@canyon289
Copy link
Member

Rebased on master for TravisCI fix

@ahartikainen
Copy link
Contributor Author

LGTM, I will merge this now.

@ahartikainen ahartikainen merged commit dcc1636 into master Jan 15, 2019
@ahartikainen ahartikainen deleted the bugfix/multichain_ppcplot branch January 15, 2019 23:04
@ColCarroll ColCarroll mentioned this pull request Feb 22, 2019
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.

3 participants