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

Clarify RasterDataset documentation for is_image and dtype #1811

Conversation

dmeaux
Copy link
Contributor

@dmeaux dmeaux commented Jan 17, 2024

The documentation for the is_image and dtype properties of RasterDataset were vague in regards to what to do for surface models, like DEMs. The changes proposed attempt to clarify how one should go about treating surface models for different circumstances when creating a custom RasterDataset.

@github-actions github-actions bot added documentation Improvements or additions to documentation datasets Geospatial or benchmark datasets labels Jan 17, 2024
docs/tutorials/custom_raster_dataset.ipynb Outdated Show resolved Hide resolved
docs/tutorials/custom_raster_dataset.ipynb Outdated Show resolved Hide resolved
torchgeo/datasets/geo.py Outdated Show resolved Hide resolved
torchgeo/datasets/geo.py Outdated Show resolved Hide resolved
torchgeo/datasets/geo.py Outdated Show resolved Hide resolved
@adamjstewart adamjstewart added this to the 0.5.2 milestone Jan 17, 2024
@adamjstewart
Copy link
Collaborator

You can also include the CSV changes if you want: "image -> DEM"

@dmeaux
Copy link
Contributor Author

dmeaux commented Jan 18, 2024

Done.

Copy link
Collaborator

@adamjstewart adamjstewart left a comment

Choose a reason for hiding this comment

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

I handled the last remaining change requests and improved the formatting a bit, hope you don't mind. Wanted to squeeze this into the next release (coming soon!)

@isaaccorley isaaccorley merged commit 1eaade2 into microsoft:main Mar 2, 2024
25 checks passed
isaaccorley pushed a commit that referenced this pull request Mar 2, 2024
* Change DEMs from mask to image (is_image=True)

* fix to revert to upstream file

* fix unused type: ignore comment

* Update torchgeo/datasets/geo.py

Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>

* Update documentation to explain is_image and dtype. Update asterdem to override dtype.

* fix linting errors

* Made comment for is_image more succint.

* change asterdem dtype back to float32 (same as RasterDataset)

* removed integer images from documentation

* change Digital Elevation Model to DEM

* Clarify is_image and dtype.
Revert DEMs to masks

* Finish reverting DEMs to masks

* address review comments

* Changed Aster Global DEM and EU-DEM Dataset types to "DEM"

* Reorganize some information

* Use better formatting

---------

Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datasets Geospatial or benchmark datasets documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants