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

Cropped Landsat example data warns in documentation due to different pixel interpretation #598

Open
rhugonnet opened this issue Aug 2, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation priority

Comments

@rhugonnet
Copy link
Contributor

The example gu.examples.get_path("everest_landsat_b4_cropped") does not have the same pixel interpretation as gu.examples.get_path("everest_landsat_b4") (likely because of the way our old crop() function was overridding the attribute to "Area" when we saved that file, see #127).
This is creating a lot of new warnings in the documentation (for instance here: https://geoutils.readthedocs.io/en/stable/core_lazy_load.html#lazy-passing-of-georeferencing-metadata).

We should probably update it to be the same interpretation to facilitate the documentation example, and update the few tests that relied on that difference to override the attribute back to "Area" before the test.

@rhugonnet rhugonnet changed the title Cropped landsat data creates warnings in documentation due to different pixel interpretation Cropped Landsat example data warns in documentation due to different pixel interpretation Aug 2, 2024
@rhugonnet rhugonnet added documentation Improvements or additions to documentation priority labels Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority
Projects
None yet
Development

No branches or pull requests

1 participant