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.

Sign up for GitHub

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

Jump to bottom

Update SRTM to 2.3 #252

Merged
merged 3 commits into from
Aug 23, 2021
Merged

Update SRTM to 2.3 #252

merged 3 commits into from
Aug 23, 2021

Conversation

krober10nd
Copy link
Collaborator

  • Update version of SRTM

Update setup.sh
940683c 
* Update version of SRTM
@krober10nd
Copy link
Collaborator Author

Probably need to grep to find out where else this older version was used to finish this PR...

@WPringle
Copy link
Collaborator

Yes, and I think we should change the srtm output filename to just srtm.nc then we don't need to keep updating that.

@WPringle
changing SRTM15+ filename to remove version number so as to avoid nee…
9123a7e 
…ding to update. Also use wildcard in wget ftp download to automatically get the newest version
@WPringle
removing unnecessary commited files
448d9e0
WPringle
WPringle approved these changes Aug 23, 2021
View reviewed changes
Copy link
Collaborator

@WPringle WPringle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed all SRTM filenames and added wildcard to download path.

@krober10nd krober10nd merged commit 07f98a9 into Projection Aug 23, 2021
@krober10nd krober10nd deleted the version_srtm branch August 23, 2021 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WPringle WPringle WPringle approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krober10nd @WPringle