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

DTM refactor and new NRW DTM provider #129

Merged
merged 12 commits into from
Jan 11, 2025

Conversation

kbrandwijk
Copy link
Collaborator

@kbrandwijk kbrandwijk commented Jan 10, 2025

Introduces NRWProvider class for handling North Rhine-Westphalia DGM1 data. Integrates data download, merge, reproject, and convert functionalities. Enhances terrain data capabilities with community-driven features.

Refactors DTM provider structure to make base provider responsible for the main flow, and reduce DTM provider implementations' responsibility to downloading tiles.

DTMProvider:

  • Implement new processing flow and structure for derived classes
  • Added Easy Mode settings to base settings class to apply to all providers
  • Added 'base instructions' that are always displayed before provider specific instructions

SRTM:

  • Added support for downloading multiple tiles
  • Adjusted to use the new derived class implementation pattern

USGS:

  • Adjusted to use the new derived class implementation pattern
  • Removed all 'custom' logic that wasn't provider specific

NRW:

  • Implemented using the new derived class implementation pattern
  • Provides DTM 1m data for Germany NRW area

Introduces NRWProvider class for handling North Rhine-Westphalia DGM1 data. Integrates data download, merge, reproject, and convert functionalities.
Enhances terrain data capabilities with community-driven features.
@iwatkot iwatkot self-assigned this Jan 10, 2025
@iwatkot iwatkot added the DTM Provider Adds new or edits existing DTM provider. label Jan 10, 2025
@kbrandwijk kbrandwijk changed the title WIP: NRW DTM provider WIP: DTM refactor and new NRW DTM provider Jan 11, 2025
@kbrandwijk kbrandwijk changed the title WIP: DTM refactor and new NRW DTM provider DTM refactor and new NRW DTM provider Jan 11, 2025
@kbrandwijk kbrandwijk changed the title DTM refactor and new NRW DTM provider WIP: DTM refactor and new NRW DTM provider Jan 11, 2025
@kbrandwijk kbrandwijk changed the title WIP: DTM refactor and new NRW DTM provider DTM refactor and new NRW DTM provider Jan 11, 2025
@iwatkot iwatkot added the enhancement New feature or request label Jan 11, 2025
@iwatkot iwatkot merged commit f3c4f9c into iwatkot:main Jan 11, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DTM Provider Adds new or edits existing DTM provider. enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants