pytorch/examples

MNIST test_loader not being used as intended (wrong)!

Open

#756 ouverte le 19 avr. 2020

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Python (9 429 forks)batch import
good first issue

Métriques du dépôt

Stars
 (21 634 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

In the MNIST code, around line 57 (I think), the following code is wrong in my opinion for data, target in test_loader: It should be replaced by for idx,(data, target) in test_loader: The original code gives an error (int has no attribute .to(device)) as originally, data is actually assigned an integer (idx value) wrongly. I have verified this fix works. The code for training is correct but surprisingly there is a mistake in test code.

Guide contributeur

Direction de recherche
Le problème se trouve dans le code d'exemple MNIST du dépôt pytorch/examples, spécifiquement dans la boucle de test autour de la ligne 57 du fichier mnist/main.py. Le bogue est que `test loader` est itéré incorrectement : `for data, target in test loader` devrait être `for idx,(data, target) in test loader`. La correction est simple et confirmée par l'auteur du problème. Pour contribuer, localisez le fichier, appliquez le changement et soumettez une pull request.
Stack technique
python
Domaine
machine learning
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90