Skip to content

Add deprecation warning to reproject docstring - #1916

Draft
JoerivanEngelen wants to merge 4 commits into
masterfrom
issue_#1905_fix_reproject
Draft

JoerivanEngelen wants to merge 4 commits into
masterfrom
issue_#1905_fix_reproject

Conversation

@JoerivanEngelen

@JoerivanEngelen JoerivanEngelen commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Fixes #1905

Description

Throw deprecationwarning when user attempts to use reproject and add some warnings in the docstring about the deprecation, as well as the resampling probably not working anymore.

Still in draft:
Probably good to verify if things really do not work anymore as they did, as from what I see rioxarray is supposed to resample. I don't think we should let people just regrid grids with this, as xugrid is much more suited for that.

Checklist

  • Links to correct issue
  • Update changelog, if changes affect users
  • PR title starts with Issue #nr, e.g. Issue #737
  • Unit tests were added
  • If feature added: Added/extended example
  • If feature added: Added feature to API documentation
  • If pixi.lock was changed: Ran pixi run generate-sbom and committed changes

@sonarqubecloud

Copy link
Copy Markdown

@JoerivanEngelen
JoerivanEngelen marked this pull request as draft September 17, 2026 15:45
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.

[Bug] - imod.prepare.reproject does not reproject to new grid size

1 participant