search

bird-house / flyingpigeon

WPS processes for climate model data, indices and extreme events
http://flyingpigeon.readthedocs.io/en/latest/
Apache License 2.0
19 stars 15 forks source link

only support Python 3.6? #270

Closed cehbrecht closed 5 years ago

cehbrecht commented 5 years ago

... if so, we can disable Python 2.7 in .travis.yml and skip the six package.

We can not support Python 3.7 yet. This needs fixes in pywps and owslib.

Zeitsperre commented 5 years ago

Given that most of the birds install an instance of Miniconda3 on build, I get the impression that system python isn't an issue. In fact, for raven, I've set our travis.yml to build language: generic to save on time. If that's the case, I imagine we can move forward with only one Python. If we want to transition away from it, we could set a date and add deprecation warnings?

nilshempelmann commented 5 years ago

@cehbrecht For the processes left over in the streamlined Flyingpigeon.v1.4.x I do not see a need for Python 2.7. We can disable 2.7 in .travis.yml

cehbrecht commented 5 years ago

@Zeitsperre the system Python is just use as an indicator for the Python version that should be used for the conda environment: https://conda.io/docs/user-guide/tasks/use-conda-with-travis-ci.html

But we still need to check different Python 3.x versions, 3.6, 3.7. But 3.7 is not supported by owslib/pywps yet.

cehbrecht commented 5 years ago

@nilshempelmann feel free to adapt .travis.yml.

nilshempelmann commented 5 years ago

py2.7 disabled in next branch

nilshempelmann commented 5 years ago

Will be fixed with https://github.com/bird-house/flyingpigeon/pull/271

nilshempelmann commented 5 years ago

DONE