Skip to content

fix for checking dem intersection over the AM #29

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
Merged
abradley60 merged 2 commits into from
Dec 4, 2025
Merged

fix for checking dem intersection over the AM #29

abradley60 merged 2 commits into from
Dec 4, 2025

Conversation

@abradley60
Copy link
Collaborator

@abradley60 abradley60 commented Dec 4, 2025 •
edited
Loading

  • Bug fix for the function that checks if a given set of bounds interests with any dem tiles
  • Over the antimeridian was not being correctly handled. For example, over the ocean crossing the antimeridian was returning hundreds of intersections for the copernicus 30m dem.
  • The fix will split AM corring bounds and check either side
  • Adding tests for the scenario where the bug was originally found

@abradley60 abradley60 requested a review from arcisad December 4, 2025 00:37
@abradley60 abradley60 merged commit 19dd2d3 into main Dec 4, 2025
3 checks passed
@abradley60 abradley60 deleted the fix/handle_am_for_dem_in_bounds branch December 4, 2025 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arcisad arcisad arcisad 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.

3 participants

@abradley60 @arcisad