clab/dynet

Concatenate batch fails if vectors are of sizes (dim,) and (dim, 1)

Open

#531 ouverte le 8 mai 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)C++ (701 forks)github user discovery
good first issuemoderate bug

Métriques du dépôt

Stars
 (3 433 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Hi! The title sums it up. If you try to concatenate d-dimensional vectors but some vectors' shape is (d,) while others is (d, 1), dynet.concatenate_batch fails. However, if you do dynet.concatenate_cols and then a reshape, everything works fine.

Guide contributeur

Direction de recherche
Examinez l'implémentation de concatenate batch pour comprendre pourquoi elle rejette les incohérences de forme entre (dim,) et (dim,1) alors que concatenate cols fonctionne. Tracez le chemin du code et identifiez la vérification de forme qui échoue.
Stack technique
python
Domaine
backendai
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.
2
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70