-
Notifications
You must be signed in to change notification settings - Fork 378
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
Add AsterGDEM dataset #404
Conversation
.. versionadded:: 0.3 | ||
""" | ||
|
||
is_image = False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing to do right now, but I think a DEM layer could be used as an input or target
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I usually like to separate the dataset name (human-readable, informative) from the class name (short, CamelCase). Otherwise, this looks great, thanks!
Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
* add astergdem dataset * add astergdem dataset * add plot method * typo * fix docs * requested changes * Update docs/api/datasets.rst Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com> * Update torchgeo/datasets/astergdem.py Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com> * split regex * split regex * split regex * regex Co-authored-by: Caleb Robinson <calebrob6@gmail.com> Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
This PR closes #94 by adding one DEM dataset, namely the Aster Global Digital Evaluation Map dataset. It can be manually downloaded, and there are more than 20,000 different tifs covering the globe. DEMs are at ~30m resolution and were taken by the Aster instrument.
Plot Example: