CF accessor adds grid mapping as coordinate
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 28/100
Línea de trabajo
Start by running the minimal example and tracing the ds.cf[standard_name] accessor alongside Dataset.assign(). Determine whether adding the grid-mapping variable as a coordinate is intended; done should include an agreed behavior for coordinate handling that avoids the shown MergeError, with a regression check for the example.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Possibly a duplicate of #357 or a discussion topic. Opinions welcomed.
Description
Since cf_xarray == 0.8.0 (#391), selecting variables by standard name with ds.cf[standard_name] results in a different set of coordinates than selecting them by their short name with ds[short_name]. This prevents re-injecting these variables in a compatible (and even the original) dataset using Dataset.assign() (and possibly other xarray merge operations). Instead trying to Dataset.assign() a cf-picked, georeferenced variable into a raises a MergeError.
Could you help me to understand if this is a design choice or a bug? In my case it comes as a breaking change and I am not exactly sure how to go about (as a cf-xarray fix or downstream in https://github.com/juseg/hyoga/issues/73). Happy to contribute as I can.
Miminal example
import xarray as xr
import cf_xarray
# make grid-mapped fake data (or use xr.open_dataset)
ds = xr.Dataset({
'tas': ((), None, {
'grid_mapping': 'crs',
'standard_name': 'air_temperature'}),
'crs': ((), None, {
'grid_mapping_name': 'latitude_longitude'})})
# ds and ds.cf variables have different coordinates
print('crs' in ds['tas'].coords) # False
print('crs' in ds.cf['air_temperature'].coords) # True
# assign a cf-picked variable (only works on cf_xarray < 0.8.0)
ds.assign(new=ds.cf['air_temperature']) # xarray.core.merge.MergeError
Error message
Traceback (most recent call last):
File "/home/julien/git/code/hyoga/bugs/cfxr08.py", line 17, in <module>
ds.assign(new=ds.cf['air_temperature']) # xarray.core.merge.MergeError
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.12/site-packages/xarray/core/dataset.py", line 6080, in assign
data.update(results)
File "/usr/lib/python3.12/site-packages/xarray/core/dataset.py", line 4946, in update
merge_result = dataset_update_method(self, other)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.12/site-packages/xarray/core/merge.py", line 1104, in dataset_update_method
return merge_core(
^^^^^^^^^^^
File "/usr/lib/python3.12/site-packages/xarray/core/merge.py", line 772, in merge_core
raise MergeError(
xarray.core.merge.MergeError: unable to determine if these variables should be coordinates or not in the merged result: {'crs'}
- Lenguaje dominante
- Python
- Estrellas
- 181
- Forks
- 51
- Merge medio
- 6 h 39 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de xarray-contrib/cf-xarray
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
xarray-contrib/cf-xarray#623 ·
-
Consider adding opt-in support for circular axes when calling `bounds_to_vertices` (related to #594)Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 38/100
xarray-contrib/cf-xarray#596 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
xarray-contrib/cf-xarray#570 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 15/100
xarray-contrib/cf-xarray#568 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
xarray-contrib/cf-xarray#566 · 2 comentarios ·
Todos los issues de xarray-contrib/cf-xarray
Issues similares
-
Update Python support to 3.15Abiertopython-version
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
bug javascript P2-medium python release:v3.1
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
adrirubio/claude-deck#546 ·
Los mantenedores suelen responder en 1 día
-
area: desktop area: website priority: P2 type: feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
appandflow/stim#3411 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 Menos de una hora Aptitud para principiantes 88/100
baptistehamon/lsapy#185 ·
Los mantenedores suelen responder en 1 día