Fixes failing tests (#113)

* Fixes failing tests

* removes python 3.5 support

* Skips conda upload if a PR

.travis.yml

@@ -10,11 +10,10 @@ os:
   - osx
 
 env:
-  - PYTHON_VERSION=3.5 HAS_GDAL=true
   - PYTHON_VERSION=3.6 HAS_GDAL=true
-  - PYTHON_VERSION=3.5 HAS_GDAL=false
   - PYTHON_VERSION=3.6 HAS_GDAL=false
-
+  - PYTHON_VERSION=3.7 HAS_GDAL=true
+  - PYTHON_VERSION=3.7 HAS_GDAL=false
 before_install:
   # We do this conditionally because it saves us some downloading if the
   # version is the same.
@@ -56,7 +55,10 @@ after_success:
   - conda config --set anaconda_upload no
   - conda build recipe -q
   - builddir=(`conda build recipe --output`)
-  - anaconda -t="$CONDA_UPLOAD_TOKEN" upload $builddir --label dev --force;
+  - |
+    if [ "$TRAVIS_PULL_REQUEST" == "false" ]; then
+      anaconda -t="$CONDA_UPLOAD_TOKEN" upload $builddir --force;
+    fi
 
   # Docs to gh-pages
   - source activate test  # Reactivate the env to have all deps installed.

environment.yml

@@ -3,8 +3,8 @@ channels:
   - conda-forge
   - usgs-astrogeology
 dependencies:
-  - libgdal 
-  - gdal 
+  - libgdal < 3 
+  - gdal < 3 
   - numpy 
   - pyproj 
   - h5py 

plio/io/io_gdal.py

@@ -191,6 +191,7 @@ class GeoDataset(object):
 
     @property
     def forward_affine(self):
+        print(self.geotransform)
         self._fa = affine.Affine.from_gdal(*self.geotransform)
         return self._fa
 
@@ -460,6 +461,7 @@ class GeoDataset(object):
                    (Latitude, Longitude) corresponding to the given (x,y).
 
         """
+
         lon, lat = self.forward_affine * (x,y)
         lon, lat, _ = self.coordinate_transformation.TransformPoint(lon, lat)
         return lat, lon

plio/io/io_yaml.py

@@ -17,7 +17,7 @@ def read_yaml(inputfile):
     """
     try:
         with open(inputfile, 'r') as f:
-            ydict = yaml.load(f)
+            ydict = yaml.safe_load(f)
     except:  # pragma: no cover
         raise IOError('Unable to load YAML file.')
     return ydict

plio/io/tests/test_io_gdal.py

@@ -62,7 +62,9 @@ class TestMercator(unittest.TestCase):
     def test_spheroid(self):
         sphere = self.dataset.spheroid
         self.assertAlmostEqual(sphere[0], 3396190.0, 6)
-        self.assertEqual(self.dataset.spheroid, (3396190.0, 3376200.0, 169.8944472236118))
+        self.assertAlmostEqual(self.dataset.spheroid[0], 3396190.0)
+        self.assertAlmostEqual(self.dataset.spheroid[1], 3376200.0)
+        self.assertAlmostEqual(self.dataset.spheroid[2], 169.8944472236118)
         self.assertAlmostEqual(sphere[1], 3376200.0, 6)
         self.assertAlmostEqual(sphere[2], 169.8944472236118, 6)
 

plio/io/tests/test_metadata.py

@@ -52,7 +52,8 @@ def test_export_to_proj4(srs_mars):
     Check that proj4 is not supporting Moon2000_Mercator
     """
     proj4 = srs_mars.ExportToProj4()
-    assert proj4 == '+proj=merc +lon_0=180 +lat_ts=0 +x_0=0 +y_0=0 +a=1737400 +b=1737400 +units=m +no_defs '
+    for element in '+proj=merc +lon_0=180 +lat_ts=0 +x_0=0 +y_0=0 +a=1737400 +b=1737400 +units=m +no_defs'.split():
+        assert element in proj4
 
 @pytest.mark.skipif(gdal is None, reason="GDAL not installed")
 def test_scale_factor(srs_mars):

plio/utils/log.py

@@ -32,7 +32,7 @@ def setup_logging(path=get_path('logging.json'),
                 config = json.load(f)
             elif logtype == '.yaml':
                 import yaml
-                config = yaml.load(f.read())
+                config = yaml.safe_load(f.read())
         logging.config.dictConfig(config)
     else:
         logging.basicConfig(level=level)