Potentially slow iteration for setting condition value units
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 150
- Forks
- 23
- Merge moyen
- 1 j 11 h
- PR mergées (30 j)
- 2
Description
As noted by @davidhassell in the review thread https://github.com/NCAS-CMS/cf-python/pull/427#discussion_r1040910208, the crux being:
if a large set is stored as a Data object, as opposed to a python built-in like a list, then iterating through it could be slow
relating to the code (currently on the lama-to-dask branch only):
with a suggested solution of:
For the Data object case we could just change the units without iterating through.
Guide de contribution
Ouvrir le guide de contribution
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.
Piste de recherche
Commencez par cf/query.py aux lignes 890-896 et par la discussion associée dans la pull request 427, en vous concentrant sur la manière dont sont traitées les valeurs de condition représentées par des objets Data. Confirmez le comportement sur la branche lama-to-dask et préservez le résultat existant de définition des unités, tout en évitant d'itérer sur de grands objets Data.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- data
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100