facebookarchive/caffe2

Accuracy requires INT32 while OneHot requires INT64

Open

#1 402 ouverte le 30 oct. 2017

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Shell (1 954 forks)batch import
enhancementhelp wanted

Métriques du dépôt

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

Description

The operator Accuracy() takes parameter labels as type core.DataType.INT32. OneHot() takes parameter indices as type core.DataType.INT64. LabelCrossEntropy() takes label as type core.DataType.INT32. It's a bit waste of time to cast the label back and forth, hope there is alignment. Thx.

Guide contributeur

Direction de recherche
Examinez le code source des opérateurs Accuracy et OneHot pour aligner les types de données. L'opérateur Accuracy attend INT32 pour les étiquettes tandis que OneHot attend INT64 pour les indices. Recherchez les définitions de type dans l'enregistrement de l'opérateur et envisagez d'en modifier un pour correspondre à l'autre pour des raisons de cohérence.
Stack technique
pythoncpp
Domaine
machine learningbackend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65