matplotlib / matplotlib/pytest-mpl
result_image redundancy
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 272
- Forks
- 53
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
In the `plugin.ImageComparison.compare_image_to_hash_library` method, during `hybrid-mode`, the `result_image` appears to be unnecessarily copied into the `summary` dictionary from the outcome of `plugin.ImageComparison.compare_image_to_baseline`:
https://github.com/matplotlib/pytest-mpl/blob/e3876180847c2d04aeda1982372074998a7f8716/pytest_mpl/plugin.py#L571-L573
i.e., the `summary['result_image']` has already been correctly set within `plugin.ImageComparison.compare_image_to_hash`:
https://github.com/matplotlib/pytest-mpl/blob/e3876180847c2d04aeda1982372074998a7f8716/pytest_mpl/plugin.py#L551-L554
If this is the case, are you happy for me to remove this behaviour?
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Línea de trabajo
Comienza en pytest_mpl/plugin.py, en ImageComparison.compare_image_to_hash_library y compare_image_to_baseline, especialmente en las líneas referenciadas alrededor de 551-554 y 571-573. Confirma cómo hybrid-mode rellena summary['result_image']; después elimina únicamente el comportamiento redundante si la asignación anterior es suficiente y verifica que el comportamiento de comparación de imágenes no cambie.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- testing
- Tipo de issue
- Refactorización
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 35/100