pyro-ppl/pyro

Enable warnings in tests/test_examples.py

Offen

#2.201 geöffnet am 03.12.2019

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (981 Forks)batch import
help wantedwarnings & errors

Repository-Metriken

Stars
 (8.211 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 10T 19h) (1 gemergte PR in 30 T)

Beschreibung

I just noticed that our usual pytest warnings filters (-Werror with a few exceptions) do not apply to the eamples/*.py when run by subprocess.check_call(), leading to some warnings raised by examples.

I tried sketching a possible solution in the example-warnings branch, but it appears difficult to specify exact warnings, and there are some possibly unrelated issues with torch_shm_manager and with the jit raising SystemError: ... returned a result with an error reset.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie sich pytest Warnfilter (z. B. Werror) auf Subprozessaufrufe übertragen lassen, die Beispielskripte ausführen. Überprüfen Sie die vorhandene Warnfilterkonfiguration in pytest und passen Sie sie für Subprozesse an. Erwägen Sie die Verwendung von Umgebungsvariablen wie PYTHONWARNINGS oder die Änderung des Subprozessaufrufs, um geeignete Flags zu übergeben.
Tech Stack
pythonpytorch
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Pythonpytest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60