Skip to content

Update clip wrapper and replace clip script - #18

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

Update clip wrapper and replace clip script#18
irm-codebase wants to merge 2 commits into
mainfrom
feature/improved-clipping

Conversation

@irm-codebase

Copy link
Copy Markdown
Contributor

Fixes #17 and some of #16

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).

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.

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