Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update WRS2 footprint shapefile URL and file name #38

Open
cgmorton opened this issue Aug 1, 2018 · 1 comment
Open

Update WRS2 footprint shapefile URL and file name #38

cgmorton opened this issue Aug 1, 2018 · 1 comment

Comments

@cgmorton
Copy link
Contributor

cgmorton commented Aug 1, 2018

Per dgketchum/Landsat578#28, we will likely need to update the WRS2 descending shapefile url in tools/download/download_footprints.py from:
https://landsat.usgs.gov/sites/default/files/documents/wrs2_descending.zip
to:
https://landsat.usgs.gov/sites/default/files/documents/WRS2_descending.zip

We will also need to update all the INIs or modify the download script so that the unzipped shapefile is renamed to match the old (all lowercase) style.

cgmorton added a commit that referenced this issue Sep 17, 2018
The URL for the WRS2 shapefile changed slightly (WRS is now uppercase in the file name).  I think it would be better to change all the INIs to expect the uppercase version of the filename, but for now, renaming the shapefile to lowercase should minimize disruption in the current workflow.  See issue #38 for more details.  This may be something we want to revisit in a 0.2.0 release.
@cgmorton cgmorton mentioned this issue Sep 17, 2018
@cgmorton
Copy link
Contributor Author

This is temporarily fixed by renaming the shapefile to the lower case version, but it would be better to eventually switch to using the original file name.

dgketchum pushed a commit to dgketchum/pymetric that referenced this issue Oct 24, 2020
The URL for the WRS2 shapefile changed slightly (WRS is now uppercase in the file name).  I think it would be better to change all the INIs to expect the uppercase version of the filename, but for now, renaming the shapefile to lowercase should minimize disruption in the current workflow.  See issue WSWUP#38 for more details.  This may be something we want to revisit in a 0.2.0 release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant