-
Notifications
You must be signed in to change notification settings - Fork 48
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
Use a requirements.txt file #4
Comments
looks good! Is there a way to distinguish between the requirements for ODYM only and those for the Jupyther notebooks? |
Well I guess the question is if we build upon Jupyter notebooks or not. |
The alternative is that users are responsible for installing Jupyther, e.g., via the Anaconda distribution. |
I will come back to this after having a look at the tutorials this week |
Hi, Based on my experience with pymrio and country_converter you should aim to get a package as soon as possible. It is not necessary that you upload that to pypi straight away, you can still just clone the repo and than do an You can also do a This will solve your requirement issue and also the need for specifying where the source code is in the settings file. Otherwise, you are reinventing the wheel in a way the user would not expect... Once you are happy with the software, it is just one command to publish it on pypi and any user can install it with the same command without cloning from github. You can check here for a minimal setup file to make a package: As I am not really involved in this software, I will unsusbscribe from these issues - however, you can still mention me with @konstantinstadler and I will have a look |
@konstantinstadler Thanks for dropping in. This is very helpful. Stefan and I had this very discussion yesterday. I like your suggestion and will read up on how to create packages. Are the packages managed in |
Yes. Perhaps the easiest way to try these things would be to clone the country converter as a running example and play with it. You can alway get rid of installed packages by |
We need a requirements.txt. The current file imports everything. I am not sure what is the purpose of this.
Would you mind if I replaced it with the following, @stefanpauliuk ?
In my case ODYM seems to work with the following (i.e.
pip freeze
output):This includes Jupyter. Without that it would be much less dependencies.
The text was updated successfully, but these errors were encountered: