facebookresearch/ParlAI

Standardize batchifying image features

Ouverte

#2 461 ouverte le 11 mars 2020

Voir sur GitHubGuide contributeur
 (4 commentaires) (1 réaction) (0 personne assignée)Python (2 089 forks)batch import
H2 2020Help WantedSmalldonotreap

Métriques du dépôt

Stars
 (10 371 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Currently, ImageSeq2seqAgent and ImagePolyencoderAgent have two separate .batchify_image_features() methods for preparing image features for encoding. The main advantage of ImagePolyencoderAgent's implementation of this method is that the images are packed into a Tensor for data_parallel. It's worth testing whether ImageSeq2seqAgent really requires a separate .batchify_image_features() method now: if not, it should be removed, and ImagePolyencoderAgent.batchify_image_features() should be moved into TorchImageAgent.

Guide contributeur

Direction de recherche
Examinez les implémentations actuelles de `batchify image features()` dans `ImageSeq2seqAgent` et `ImagePolyencoderAgent`. Déterminez si la méthode de `ImageSeq2seqAgent` peut être remplacée par la version de `ImagePolyencoderAgent`. Si oui, proposez de déplacer la méthode unifiée vers `TorchImageAgent` et de tester son exactitude.
Stack technique
python
Domaine
backendai
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonPyTorch
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75