-
Notifications
You must be signed in to change notification settings - Fork 97
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
Code still up to date? #18
Comments
Hi, This is fixed in #14, sorry for the time it took you to fix. @anazalea could you merge this and update the pypi version ? It would permit to use pysankey as a dependency instead of copy pasting the code in django survey and report. Thank you in advance. |
I released a beta with the last working code as pySankeyBeta. You can install it with :
Also fixed a bug preventing to release current master branch on Pypi here : d53bfd3 |
@Pierre-Sassoulas Could you help me with the imports? After installing, I tried importing like this as recommended: However, this didn't work so I tried installing via I didn't manage to import it after all, any idea what's happening? |
Seeing the error you get would be helpful to be able to know what is happenning exactly. Can you |
@Pierre-Sassoulas Cannot |
Try removing |
Have remove pySankey and install pySankeyBeta and run example, but ...... import matplotlib.pyplot as plt ModuleNotFoundError Traceback (most recent call last) ModuleNotFoundError: No module named 'pySankeyBeta' |
The import in the code should stay the same: 'from pysankey' :) only the pypi name is different |
I installed the package via pip install and had to change a lot in order to make it work, wherefore I wonder wether this is the most up to date manual?
Issues -> fixes I had to do:
from pysankey import sankey -> from pySankey import sankey (otherwise cant find module)
sankey(df... -> sankey.sankey(.... (since we do not import the function directly)
figureName -> figure_name (otherwise it doesn't work)
So you may want to update the example?
Thank you
The text was updated successfully, but these errors were encountered: