set_options is applied on init, not enter
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Start in cf_xarray/options.py, linked from the issue, and inspect the set_options context-manager lifecycle. Add a test matching the ds_1 and ds_2 example, then run the relevant test suite; done means each dataset uses its corresponding options only while its context manager is active.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Really nice stuff! I'm not impacted by this, but I noticed a bug. In the following example, I would expect corresponding options to be applied to operations on their corresponding datasets. Currently, as I understand set_options, ds_2_options will be used for both ds_1 calls, and neither context manager will be used for the ds_2 call.
ds_1_options = set_options(...)
ds_2_options = set_options(...)
ds_1_cf.<something>
with ds_1_options:
ds_1.cf.<something>
with ds_2_options:
ds_2.cf.<something>
I believe the fix would involve writing a test like the above example and moving the init code into enter.
- Lenguaje dominante
- Python
- Estrellas
- 183
- 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
-
effort: low prio: low ready
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
digitalfabrik/integreat-cms#4616 ·
Los mantenedores suelen responder en 5 días
-
bug difficulty:2 good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
MakazhanAlpamys/Soup#1865 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
FinanceFlash/unvibecode#233 ·
Los mantenedores suelen responder en 1 día
-
area: agents area: docs good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
openapi/gknowledge#23 ·
-
useless-suppression reported for deprecated-* messages that should be exemptPosiblemente ocupada @adityasinha-ghub la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
pylint-dev/pylint#11656 ·
Los mantenedores suelen responder en 1 día