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

Lat/Lon Datastore dict causes issues with xarray backend #117

Closed
EricEngle-NOAA opened this issue Jan 3, 2024 · 0 comments · Fixed by #118
Closed

Lat/Lon Datastore dict causes issues with xarray backend #117

EricEngle-NOAA opened this issue Jan 3, 2024 · 0 comments · Fixed by #118
Assignees
Labels
bug Something isn't working

Comments

@EricEngle-NOAA
Copy link
Collaborator

In v2.1, the latitude, longitude datastore was modified to store values for each unique grid as a dictionary with keys 'latitude' and 'longitude' and potentially other data as needed. Some of the logic added in that modification has introduced a bug when using the xarray backend.

@EricEngle-NOAA EricEngle-NOAA added the bug Something isn't working label Jan 3, 2024
@EricEngle-NOAA EricEngle-NOAA self-assigned this Jan 3, 2024
EricEngle-NOAA added a commit that referenced this issue Jan 3, 2024
This commit fixes an issue where the _latlon_datastore, now holding a dictionary
instead of tuples would raise a KeyError when using the xarray backend.

This commit references issue #117
@EricEngle-NOAA EricEngle-NOAA linked a pull request Jan 3, 2024 that will close this issue
EricEngle-NOAA added a commit that referenced this issue Jan 5, 2024
This commit fixes an issue where the _latlon_datastore, now holding a dictionary
instead of tuples would raise a KeyError when using the xarray backend.

This commit references issue #117

Co-authored-by: Eric Engle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant