search

sgrieve / GeoMeta

Collaborations workshop hackday 2018 project
MIT License
2 stars 2 forks source link

Include full dependencies #27

Open ageorgou opened 6 years ago

bsipocz commented 6 years ago

I think this is an overkill for now, many of these are testing or dev requirement, and the version dependencies on packages can be more liberal.

bsipocz commented 6 years ago

I've added extra tests for different numpy versions, etc, so we definitley can remove the version restriction from that one.

ageorgou commented 6 years ago

The issue was more with having an exhaustive list of dependencies - we were having problems with e.g. rasterio not being installed correctly because numpy was missing.

I agree it's not of massive importance though.

bsipocz commented 6 years ago

hmm, but then that's an upstream rasterio issue, this really should have been dealt by pip, etc. Anyway, if this project would go forward I suggest to package it up properly and any of these dependency issues can be dealt inside setup.py setup.cfg, etc.