Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make more use of xarray's decode_coords='all' #3665

Open
dopplershift opened this issue Oct 25, 2024 · 0 comments
Open

Make more use of xarray's decode_coords='all' #3665

dopplershift opened this issue Oct 25, 2024 · 0 comments
Labels
Area: Xarray Pertains to xarray integration Type: Enhancement Enhancement to existing functionality

Comments

@dopplershift
Copy link
Member

In pydata/xarray#2844, the option decode_coords='all' was added, which handles turning the appropriate grid_mapping variables into coordinates at dataset load time. This came along after we built up a bunch of our xarray integration, so there are some things I wonder now:

  • Could we deprecate our metpy_crs coordinate in favor of relying on decode_coords='all' and doing everything else we need with projections on-demand? Might be some issues with our inference of lat/lon projections.
  • The need to work across multiple, non-coordinate variables was the main motivation for the parse_cf() approach in the beginning. Could we deprecate parse_cf() entirely and do everything else on demand? The "special" handling for GOES coordinates is still something that might be tricky.

#3664 is handling making sure we work with this option at all, since it breaks our own finding of grid_mapping in parse_cf().

cc @jthielen

@dopplershift dopplershift added Area: Xarray Pertains to xarray integration Type: Enhancement Enhancement to existing functionality labels Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Xarray Pertains to xarray integration Type: Enhancement Enhancement to existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant