search

nasa-gibs / onearth

High-performance web services for tiled raster imagery and vector tiles
Apache License 2.0
178 stars 47 forks source link

Python 3 updates #139

Closed jamesfwood closed 4 years ago

jamesfwood commented 4 years ago

Should update all python programs to python3. Passing all tests

mcechini commented 4 years ago

For our operations scripts, I've been going with #!/usr/bin/env python vs #!/usr/bin/python3 and assuming that it's up to the user to ensure that they have created an environment where python is a 3.x version. This may be a little more flexible because we run these scripts in Centos 7, and OSX, and maybe elsewhere where /usr/bin/python3 may not be what I want to be used.

mike-mcgann commented 4 years ago

Cumulus doesn't play nice and forcibly sets the python link against the will of the user and the OS. So I think it is better to explicitly reference python3.

mcechini commented 4 years ago

Then perhaps #!/usr/bin/env python3 ? Which I do see in a number of the python files already.

mike-mcgann commented 4 years ago

Yes