search

ContinuumIO / elm

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

Consider using six for Python version flexibilty #148

Closed PeterDSteinberg closed 7 years ago

PeterDSteinberg commented 7 years ago

Can we use six to simplify support for dif't Python versions?

Address this earth-env issue 6 and then reconsider changes to support Python 2 vs 3 that have already been made in elm PR #141 (most of the changes in that large PR are related to the elm.readers subpackage refactor PR 1 not necessarily Python version flexibility).

PeterDSteinberg commented 7 years ago

Also with this issue we should consider a build matrix in Travis CI for Python versions / env vars to test Python 2 and 3 versions.

PeterDSteinberg commented 7 years ago

This relates to #154

PeterDSteinberg commented 7 years ago

One of my comments above relates to build matrix for Travis CI. It currently is set to build 2.7, 3.5, 3.6 Python in Travis via env variables build matrix (PR #141), but 2.7 and 3.6 are failing.

gbrener commented 7 years ago

Closing as of PR #168