OWI-Lab / owimetadatabase-preprocessor

Basic processing of the data from owimetadatabase
https://owi-lab.github.io/owimetadatabase-preprocessor/
GNU General Public License v3.0
2 stars 1 forks source link

Dependency to scipy motivated? #14

Closed wweijtje closed 8 months ago

wweijtje commented 8 months ago

I always try to avoid dependencies to scipy when there is a valid alternative. I notice often scipy is imported for just a single function where often an equivalent (most often within numpy) exists.

The reason is that scipy comes in at a whopping 45MB.

image

Of course there are definitely functions that require scipy and if it is not just one, then it is ok. But is a dependency I always hunt down :-)

arsmlnkv commented 8 months ago

It is required for the groundhog package which is required for soil stuff. I guess in the near future, when we split the package, it can be avoided for the basic functionality package like geometry/locations.

arsmlnkv commented 8 months ago

I removed this dependency from the latest stable version (without soil functionality). When it will be realised in full, I will return it.