Add read support for Zarr spatial and proj conventions#900
Closed
emmanuelmathot wants to merge 7 commits intocorteva:masterfrom
Closed
Add read support for Zarr spatial and proj conventions#900emmanuelmathot wants to merge 7 commits intocorteva:masterfrom
emmanuelmathot wants to merge 7 commits intocorteva:masterfrom
Conversation
- Introduced a new `Convention` enum for managing geospatial metadata conventions. - Added CF convention support in the `_convention` module with functions for reading and writing CRS and transforms. - Updated `_options` to include a convention option. - Modified `XRasterBase` to utilize the convention setting for spatial dimension detection and CRS handling. - Implemented unit tests for CF convention functionalities and options.
…tion logic to `_convention/cf.py` module DOC: Update documentation to include `rioxarray.enum` module with members and inheritance
This is the first in a series of PRs splitting corteva#883 as requested by maintainers. **Changes:** - Add `Convention` enum with `CF` value - Add `convention` option to `set_options()` - Create cf.py module with extracted CF read/write logic - Refactor rioxarray.py to use the new CF module This PR establishes the framework for supporting multiple geospatial metadata conventions. Future PRs will add Zarr convention support. --- - [ ] Closes corteva#883 (partial - first of series) - [x] Tests added - [] Fully documented, including history.rst for all changes and rioxarray.rst for new API
…ion and unit tests
3 tasks
snowman2
reviewed
Feb 3, 2026
| if convention_check and not has_convention_declared(attrs, "proj:"): | ||
| return None | ||
|
|
||
| for proj_attr, parser in [ |
Member
There was a problem hiding this comment.
crs_from_user_input should work for all if the inputs. No need for individual parsers.
Contributor
Author
|
superseeded by #905 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the second in a series of PRs splitting #883 as requested by maintainers. Depends on #899.
Changes:
Convention.Zarrenum value_convention/zarr.pymodule with read functions for Zarr conventionsset_options(convention=Convention.Zarr))proj:wkt2,proj:code,proj:projjsonfor CRS readingspatial:transform,spatial:dimensionsfor transform and dimension readingReviewer feedback addressed:
CRS.from_user_input()instead of explicit PROJJSON handling