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

Raster.intersection does not raise error if rasters in different projection #306

Closed
adehecq opened this issue Sep 13, 2022 · 1 comment · Fixed by #307
Closed

Raster.intersection does not raise error if rasters in different projection #306

adehecq opened this issue Sep 13, 2022 · 1 comment · Fixed by #307
Assignees
@adehecq
Labels
bug Something isn't working

Comments

@adehecq
Copy link
Member

adehecq commented Sep 13, 2022

Currently, Raster.intersection is not implemented when the input rasters have a different projection and it does not raise an error when that occurs.

geoutils/geoutils/georaster/raster.py

Line 1669 in 96c432c

def intersection(self, rst: str | Raster) -> tuple[float, float, float, float]:

See #272 for a quick fix.
@adehecq adehecq added the bug Something isn't working label Sep 13, 2022
@adehecq adehecq self-assigned this Sep 13, 2022
@adehecq adehecq mentioned this issue Sep 13, 2022
Merged
3 tasks
@adehecq adehecq linked a pull request Sep 15, 2022 that will close this issue
Improve loading different rasters and handling different CRS in Raster #307
Merged
3 tasks
@adehecq
Copy link
Member Author

adehecq commented Sep 15, 2022

Closed via #307

@adehecq adehecq closed this as completed Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adehecq adehecq

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve loading different rasters and handling different CRS in Raster adehecq/GeoUtils
1 participant
@adehecq