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.

Sign up for GitHub

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

Jump to bottom

Add Sankey diagram generator for visualizing probability splits #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Sankey diagram generator for visualizing probability splits #10

wants to merge 1 commit into from

Conversation

stefanv
Copy link
Collaborator

@stefanv stefanv commented Jan 5, 2021

sankey

@matthew-brett
Copy link
Member

Sorry to be lazy - but to what extent is the code tied to the example diagram? I mean - if I was drawing such a diagram, I would draw it differently, but, does that matter in terms of the code here? Or is it sufficiently general that the details of this diagram don't matter? I've attached something more like what I would have drawn, in case it's relevant

image

@stefanv
Copy link
Collaborator Author

stefanv commented Jan 6, 2021

The advantage of this diagram is that it contains all the numbers you need for calculating Bayes. It should not be too hard to make it look like yours, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stefanv @matthew-brett