set_options is applied on init, not enter
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Python
- Étoiles
- 181
- Forks
- 51
- Merge moyen
- 3 h 41 min
- PR mergées (30 j)
- 2
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de xarray-contrib/cf-xarray
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
xarray-contrib/cf-xarray#623 ·
-
Consider adding opt-in support for circular axes when calling `bounds_to_vertices` (related to #594)Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 38/100
xarray-contrib/cf-xarray#596 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
xarray-contrib/cf-xarray#570 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 15/100
xarray-contrib/cf-xarray#568 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
xarray-contrib/cf-xarray#566 · 2 commentaires ·
Toutes les issues de xarray-contrib/cf-xarray
Issues similaires
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
epam/ai-dial-quickapps-backend#628 ·
Les mainteneurs répondent en général sous 2 jours
-
0xlau.dev 已失效，切换成 timlau.meOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 69/100
timqian/chinese-independent-blogs#2235 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
eclipse-score/coverage_tool#27 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
bojieli/ai-agent-book#1174 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
RedHatQE/mtv-api-tests#721 ·
Les mainteneurs répondent en général sous 1 jour