Consider replacing attrs with msgspec
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- cli, performance
Línea de trabajo
Empieza por localizar las clases de datos de attrs utilizadas por las rutas de CLI y procesamiento de datasets; el issue no nombra archivos ni puntos de entrada específicos. Mide la capacidad de respuesta de la importación de CLI y el tiempo de ejecución total con un dataset grande; después compara attrs con msgspec.Struct y documenta si la mejora de rendimiento justifica el cambio en la legibilidad.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
As this project gets closer to usability, performance on large datasets is going to become relevant. I suspect that attrs is not a bottleneck, but we do instantiate a large number of attrs dataclasses. We should evaluate whether switching to msgspec.Struct offers nontrivial performance improvements.
I would say the main two relevant measures would be:
- CLI responsiveness. If importing is slow the CLI will feel sluggish, even if we're talking a one-time cost of a couple hundred ms.
- Total runtime on a large dataset. 5-10% speedup is probably worth it. <5% if the code becomes simpler to read.
- Lenguaje dominante
- Python
- Estrellas
- 10
- Forks
- 7
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 bids-standard/python-validator
-
Add a structured issues model and emit findings from the filename validatorPosiblemente ocupada @karellopez la tomó hace 81 días. Abiertoenhancement
bids-standard/python-validator#85 · 1 comentario · 1 asignado ·
-
Populate associations contextQuizá libre de nuevo @ubdbra001 la tomó hace 115 días y no hay ningún pull request abierto. Abierto
bids-standard/python-validator#82 · 1 comentario · 1 asignado ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Validate DerivativesPosiblemente ocupada @bendhouseart la tomó hace 298 días. Abiertoenhancement
bids-standard/python-validator#62 · 1 asignado ·
-
Populate tiff contextAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de bids-standard/python-validator
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
NousResearch/hermes-agent#147104 ·
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Document new required `shared-org-key` input for `obs-aw-dependency-review`Posiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
bot-triaged module: inductor oncall: pt2 topic: fuzzer triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
getchaintips leaves out the active tip once a header is staged on it, where Core always lists itAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
btclib-org/btclib-node#2158 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
patrick-hudson/put.gd-desktop#979 · 1 comentario ·
Los mantenedores suelen responder en 1 día