Skip to content

Update clip wrapper and standardise CRS handling - #18

Open
irm-codebase wants to merge 6 commits into
mainfrom
feature/improved-clipping
Open

Update clip wrapper and standardise CRS handling#18
irm-codebase wants to merge 6 commits into
mainfrom
feature/improved-clipping

Conversation

@irm-codebase

@irm-codebase irm-codebase commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Fixes #17, #16, #19

Summary of changes in this pull request

  • updated the clip wrapper version, and switched to using it for local files. This makes one script unnecessary.
  • Small tune up to naming of configuration (slope -> slope_deg)
  • Set flooded areas as NOT_SUITABLE by default (these are mostly swamps, wetlands or estuaries).
  • Standardisation to EPSG:4326 is explicit in settings.yaml. The implementation in minimal on purpose. This could be made fully flexible later.

Reviewer checklist

  • There are no pip dependencies in the module's environment files (workflow/envs/).
  • All rules use pathvars (e.g., <results>) in their inputs and outputs.
  • The integration test-suite is successful, including:
    • pre-commit.ci tests pass.
    • tests pass for all relevant OS configurations (linux, osx, windows).
  • Module documentation is up-to-date, including:
    • INTERFACE.yaml mentions all relevant pathvars and wildcards.
    • README.md describes how to use the module and has the necessary citations.

@irm-codebase irm-codebase changed the title Update clip wrapper and replace clip script Update clip wrapper and standardise CRS handling Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update clip wrapper to v9.14.0/geo/rasterio/clip

1 participant