pyro-ppl/pyro

Enable warnings in tests/test_examples.py

Ouverte

#2 201 ouverte le 3 déc. 2019

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Python (981 forks)batch import
help wantedwarnings & errors

Métriques du dépôt

Stars
 (8 211 étoiles)
Métriques de merge PR
 (Merge moyen 10j 19h) (1 PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez comment propager les filtres d'avertissement pytest (par ex. Werror) aux appels de sous processus qui exécutent des scripts d'exemple. Examinez la configuration existante des filtres d'avertissement dans pytest et adaptez la pour le sous processus. Envisagez d'utiliser des variables d'environnement comme PYTHONWARNINGS ou de modifier l'appel de sous processus pour passer les indicateurs appropriés.
Stack technique
pythonpytorch
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60