-
Notifications
You must be signed in to change notification settings - Fork 5
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
can't import astetik without internet connection #32
Comments
Ran into this today. Would it be possible to add a |
Looks like this is already fixed: /Users/mikko/Documents/GitHub/astetik/astetik/utils/country_code.py It is in fact loading it from extras. If it for some reason throws an error (I tried on my local machine and it didn't, only then it tries to download from online). I've move the respective imports inside the function, so they should not load unless the plot itself is loaded? If there is still issue, could add another exception which simply does nothing (if online also fails). |
The file is there in git, but I don't think it makes it into the source package in PyPi.org. I downloaded the latest package from PyPI.org: https://pypi.org/project/astetik/#files and the countries.csv is not included in extras. I'm not an expert in Python packaging, but perhaps there's some changes that need to be added to setup.py to include non .py files? https://python-packaging.readthedocs.io/en/latest/non-code-files.html |
Nevermind... I'm clearly not an expert in packaging.. The setup.py has I'm just curious why the PyPi.org package doesn't have the data. When I cloned the repo and installed manually, everything worked as expected. |
I am still affected by this issue. I get KeyError: 'astetik\plots\..\extras\countries both when installing from pip and when runing setup.py The weird thing is that the files seems to be there in the .egg file, so I do no know what is going on |
This should now go away finally :) |
Excelent |
It is so. Now it's possible to |
When doing
import astetik
the issue is with countries.csv. Will have to look for a better way to do this.The text was updated successfully, but these errors were encountered: