matplotlib / matplotlib/pytest-mpl

Paramater `remove_text` in `mpl_image_compare` decorator does not remove axis labels

Offen
#120 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
272
Forks
53
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

When choosing remove_text=True I would have assumed that the text for axis labels was removed along with the axis ticks and any titles. But it doesn't seem to do that.

The code uses remove_ticks_and_titles from matplotlib and that does not appear to remove the axis labels (See matplotlib documentation).

I'd propose updating the remove_text conditional as follows:


if remove_text:
    remove_ticks_and_titles(fig)
    for ax in fig.get_axes():
        ax.set_xlabel = ""
        ax.set_ylabel = ""

If this seems reasonable, I can submit a PR.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, den Decorator mpl_image_compare und dessen remove_text-Bedingung zu lokalisieren, und untersuche dann, wie sich der bestehende Aufruf von remove_ticks_and_titles auf Achsenbeschriftungen auswirkt. Überprüfe das Verhalten mit einem Bildvergleichstest und betrachte das vorgeschlagene Verhalten zum Entfernen von Beschriftungen als umgesetzt, wenn remove_text=True sowohl Achsenbeschriftungen als auch Ticks und Titel ausschließt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
testing
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.