oemof / demandlib

Creating heat and power demand profiles from annual values.

https://oemof.org

MIT License

56 stars 38 forks source link

Examples depend on matplotlib but this isn't mentioned as requirement #4

Closed gplssm closed 7 years ago

gplssm commented 7 years ago

I suggest to keep it easy and add matplotlib to the requirements. @oemof/demandlib what do you think?

gplssm commented 7 years ago

5

uvchik commented 7 years ago

We lately had some problems installing matplotlib via pip. More requirements do not make it necessarily easier for us to maintain the package.

I would agree adding it by now but removing it immediately if problems occur again.

I agree that I makes it easier for new users running the example out of the box.

gplssm commented 7 years ago

I agree with your thoughts!

Is the matplotlib problem you mentioned related with having installed matplotlib twice? Via

sudo apt-get install python3-matplotlib

and

pip3 install matplotlib

At least in a virtualenv this is not a problem (for me).

uvchik commented 7 years ago

No, in the past pip install matplotlib just failed. But let's consider that it is a quite popular package and such problems will not occur often. So for me its fine until we know better.

gplssm commented 7 years ago

Closed by 546ef54fe3b1a43de3a478eba698e6788bd050d9