ContinuumIO / elm

Phase I & part of Phase II of NASA SBIR - Parallel Machine Learning on Satellite Data
http://ensemble-learning-models.readthedocs.io
43 stars 27 forks source link

Consider whether to support NumPy versions before 1.11 #167

Closed gbrener closed 6 years ago

gbrener commented 7 years ago

Currently, cartopy and datashader depend on NumPy 1.11. Should we consider making cartopy and datashader optional dependencies so that we can be compatible with previous NumPy versions?

PeterDSteinberg commented 7 years ago

Is the idea that we would make cartopy and datashader optional in the recipe, and then when we write up documentation we tell people that they may want to do conda install $CHANNEL_ARGUMENTS elm earthio cartopy datashader? If it is that simple, it sounds reasonable to me to do it that way.

gbrener commented 7 years ago

Yeah, that's one approach. There's also support for optional dependencies in conda being added in version 4.4: https://github.com/conda/conda/issues/1696 . We might be able to take advantage of that once it's released. Either way, I think we might want to revisit this question down the road, if we end up having users who need it.

PeterDSteinberg commented 7 years ago

Let's revisit later in Phase II. I'll add the deferred label