`--merge` should memoize the results for performance
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- ruby
- Área
- cli, performance
Línea de trabajo
Start by locating the --merge implementation and tracing how ancestor locales are merged for each locale. Compare the current traversal with the proposed breadth-first approach from root, then verify that shared ancestor data is computed once and the merged output remains unchanged. The issue names no specific files or tests.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
--merge currently takes a long time, needlessly.
Instead of iterating over each locale in turn and merging in the ancestor locales each time:
en-CA -> en -> root
en-GB -> en -> root
en-US -> en -> root
All of these use the en -> root, which is the same data, so there is no need to recompute those for each child locale.
Instead, you could iterate over the graph of locales breadth-first starting at the root locale, then cache the results for use in the other locales.
(Of course, this might not be worth doing as the whole concept of --merge is likely to change. I just wanted to capture this potential optimization here)
- Lenguaje dominante
- Ruby
- Estrellas
- 78
- Forks
- 18
- Merge medio
- 56 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. 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 ruby-i18n/ruby-cldr
-
Releasing `ruby-cldr`Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
-
Add SubdivisionContainmentAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Replace `jeweler`Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Todos los issues de ruby-i18n/ruby-cldr
Issues similares
-
P2 testing
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
performance
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Blame view link labels are wrongAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
openSUSE/open-build-service#20338 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
DB上でコメント本文がNULLを許容しているAbiertoバグ
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día