matplotlib / matplotlib/pytest-mpl
result_image redundancy
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 272
- Forks
- 53
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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:
i.e., the summary['result_image'] has already been correctly set within plugin.ImageComparison.compare_image_to_hash:
If this is the case, are you happy for me to remove this behaviour?
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in pytest_mpl/plugin.py bei ImageComparison.compare_image_to_hash_library und compare_image_to_baseline, insbesondere bei den referenzierten Zeilen um 551-554 und 571-573. Bestätige, wie hybrid-mode summary['result_image'] befüllt, entferne dann nur das redundante Verhalten, falls die frühere Zuweisung ausreicht, und überprüfe, dass das Verhalten des Bildvergleichs unverändert bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- testing
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100