Skip to content

Comments

ENH: Load and write RPCs #837#890

Merged
snowman2 merged 7 commits intocorteva:masterfrom
remi-braun:store_rpcs
Jan 22, 2026
Merged

ENH: Load and write RPCs #837#890
snowman2 merged 7 commits intocorteva:masterfrom
remi-braun:store_rpcs

Conversation

@remi-braun
Copy link
Contributor

Co-authored-by: Alan D. Snow <alansnow21@gmail.com>
@snowman2 snowman2 added the enhancement New feature or request label Jan 16, 2026
@snowman2 snowman2 added this to the 0.21.0 milestone Jan 16, 2026
@snowman2
Copy link
Member

Run micromamba run -n test pylint rioxarray/
************* Module rioxarray.raster_array
rioxarray/raster_array.py:392:4: R0914: Too many local variables (21/20) (too-many-locals)
rioxarray/raster_array.py:392:4: R0912: Too many branches (13/12) (too-many-branches)
************* Module rioxarray.raster_writer
rioxarray/raster_writer.py:230:4: R0912: Too many branches (13/12) (too-many-branches)

@remi-braun
Copy link
Contributor Author

@snowman2 Do you know how to fix these ? I mean i cannot remove the ifs in to_raster and reproject 🤔

@remi-braun
Copy link
Contributor Author

rioxarray/raster_array.py:1:0: C0302: Too many lines in module (1272/1250) (too-many-lines)

Should I split raster_array?

@snowman2
Copy link
Member

Should I split raster_array?

I think this MR is good as it is and adding more complexity is not necessary. I will merge it as is and deal with it separately. Thanks!

@snowman2 snowman2 changed the title Load and write RPCs #837 ENH: Load and write RPCs #837 Jan 22, 2026
@snowman2 snowman2 merged commit a1b211c into corteva:master Jan 22, 2026
11 of 22 checks passed
@snowman2
Copy link
Member

Thanks @remi-braun 👍

@remi-braun
Copy link
Contributor Author

Thank you for your support!

@remi-braun remi-braun deleted the store_rpcs branch January 22, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ENH: Storing RPCs in xarray

2 participants