search

Applied-GeoSolutions / gips

Geospatial Image Processing System
GNU General Public License v3.0
17 stars 5 forks source link

landsat get_geometry tileid #451

Closed ircwaves closed 6 years ago

ircwaves commented 6 years ago

I didn't catch this problem in our discussion/review of the issue, but I had to revert the change of landsatAsset's self.tile back to a string. This change could happen down the line, but more would need to go along with it because it would break compatibility for any existing archive in the following way:

  1. an existing archive has a file 
    /var/gips-test-data/landsat/tiles/012030/2017213/012030_2017213_LC8_ndvi-toa.tif
  2. after this change, GIPS would look for the file at 
    /var/gips-test-data/landsat/tiles/12030/2017213/12030_2017213_LC8_ndvi-toa.tif
  3. users would not be able to export existing data.
ircwaves commented 6 years ago

I thought saying "fixes #issuenumber" would close this...