Merge pull request #818 from girder/patch/geospatial-attr

Add `geospatial` property
girder · Jul 29, 2022 · 4d3ba5e · 4d3ba5e
2 parents e6cb017 + a026d9e
commit 4d3ba5e
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 8 deletions.
diff --git a/large_image/tilesource/__init__.py b/large_image/tilesource/__init__.py
@@ -81,7 +81,7 @@ def getSortedSourceList(availableSources, pathOrUri, *args, **kwargs):
     isLargeImageUri = str(pathOrUri).startswith('large_image://')
     extensions = [ext.lower() for ext in os.path.basename(uriWithoutProtocol).split('.')[1:]]
     properties = {
-        'geospatial': isGeospatial(pathOrUri),
+        '_geospatial_source': isGeospatial(pathOrUri),
     }
     sourceList = []
     for sourceName in availableSources:

diff --git a/large_image/tilesource/base.py b/large_image/tilesource/base.py
@@ -44,6 +44,8 @@ class TileSource:
         None: SourcePriority.FALLBACK
     }
 
+    geospatial = False
+
     def __init__(self, encoding='JPEG', jpegQuality=95, jpegSubsampling=0,
                  tiffCompression='raw', edge=False, style=None, *args,
                  **kwargs):

diff --git a/sources/gdal/large_image_source_gdal/__init__.py b/sources/gdal/large_image_source_gdal/__init__.py
@@ -115,7 +115,7 @@ class GDALFileTileSource(FileTileSource, metaclass=LruCacheMetaclass):
         'image/tiff': SourcePriority.LOW,
         'image/x-tiff': SourcePriority.LOW,
     }
-    geospatial = True
+    _geospatial_source = True
 
     def __init__(self, path, projection=None, unitsPerPixel=None, **kwargs):
         """
@@ -601,14 +601,21 @@ def getBandInformation(self, statistics=True, dataset=None, **kwargs):
             self._bandInfo = infoSet
         return self._bandInfo
 
+    @property
+    def geospatial(self):
+        """
+        This is true if the source has geospatial information.
+        """
+        return bool(
+            self.dataset.GetProjection() or
+            (self.dataset.GetGCPProjection() and self.dataset.GetGCPs()) or
+            self.dataset.GetGeoTransform(can_return_null=True) or
+            hasattr(self, '_netcdf'))
+
     def getMetadata(self):
         with self._getDatasetLock:
             metadata = {
-                'geospatial': bool(
-                    self.dataset.GetProjection() or
-                    (self.dataset.GetGCPProjection() and self.dataset.GetGCPs()) or
-                    self.dataset.GetGeoTransform(can_return_null=True) or
-                    hasattr(self, '_netcdf')),
+                'geospatial': self.geospatial,
                 'levels': self.levels,
                 'sizeX': self.sizeX,
                 'sizeY': self.sizeY,

diff --git a/sources/mapnik/large_image_source_mapnik/__init__.py b/sources/mapnik/large_image_source_mapnik/__init__.py
@@ -71,7 +71,6 @@ class MapnikFileTileSource(GDALFileTileSource, metaclass=LruCacheMetaclass):
         'image/tiff': SourcePriority.LOWER,
         'image/x-tiff': SourcePriority.LOWER,
     }
-    geospatial = True
 
     def __init__(self, path, projection=None, unitsPerPixel=None, **kwargs):
         """