Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 ahartikainen force-pushed the bugfix/multichain_ppcplot branch from 3711d6f to 0e2277e Compare January 13, 2019 21:30
@ahartikainen
Copy link
Contributor Author

Now the scatter is more flexible

Data count 30 & 10

image

Data count 30 & 1000

image

@ahartikainen ahartikainen force-pushed the bugfix/multichain_ppcplot branch from 44169a1 to a02d401 Compare January 14, 2019 07:52
@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 canyon289 force-pushed the bugfix/multichain_ppcplot branch from 1a2f118 to 1cba922 Compare January 15, 2019 06:45
@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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants