-
Notifications
You must be signed in to change notification settings - Fork 8
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
Is it possible to add option to use three columns? #9
Comments
It's probably a big change and even a completely different function to implement than the existing one, but it would be a nice feature indeed. |
Before I realised there were other active forks (sorry!), I started expanding the scope of the original pysankey code here: https://github.com/AUMAG/ausankey/ This allows multiple “stages” or columns of data to draw flows between as well as some additional options for formatting and labelling. |
Mind opening a pull request to reconcile the two forks ? |
@Pierre-Sassoulas — sadly I didn’t do my research before starting my fork (honestly I didn’t go into it with a plan to develop a whole thing, just to tinker — and then things got out of hand) Because I started from the original repo, we diverged from the start, and the code is almost unrecognisable now. Some of the interfaces are also different (I wasn’t worried about backwards compatibility). So I’m not sure a pull request would even make sense at this point… |
If it's a new function entirely we can put it alongside the other one. I was thinking of doing a v2 at some point but yeah retro-compat was a high priority (to be able to say "use this it fixes the issue" without bringing a migration into the mix). |
to have multi-categories flow?
The text was updated successfully, but these errors were encountered: