NCAS-CMS / NCAS-CMS/cf-python

Potentially slow iteration for setting condition value units

Abierto
#511 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

dask performance
Lenguaje dominante
Python
Estrellas
150
Forks
23
Merge medio
1 d 11 h
PR fusionados (30 d)
2

Descripción

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):

https://github.com/NCAS-CMS/cf-python/blob/f17e08ee69b271c3b595131ef93d25f4bb3c2395/cf/query.py#L890-L896

with a suggested solution of:

For the Data object case we could just change the units without iterating through.

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con cf/query.py en las líneas 890-896 y la discusión relacionada en el pull request 427, centrándote en cómo se manejan los valores de condición representados por objetos Data. Confirma el comportamiento en la rama lama-to-dask y conserva el resultado existente de configuración de unidades, evitando al mismo tiempo iterar sobre objetos Data grandes.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
data
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.