SINr-Embeddings / sinr

The SINr approach to train interpretable word and graph embeddings

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Bug get_dimension_stereotypes

SimonGuillot opened this issue · comments

image

La fonction get_dimension_stereotypes a un soucis de matching entre les indices des dimensions dans le dictionnaire du modèle et les indices dans dimensions dans les vecteurs.
Par exemple, pour la dimension indicée "2" dans le dictionnaire, on voit qu'elle est à la 7e position dans la liste.

Possiblement cela est du au filtrage du modèle ?

Les dimensions enlevées par le filtrage prenant la valeur -1 avec la numérotation continuant pour les dimensions conservées.
Cependant, la fonction get_dimensions_stereotypes a l'air de fonctionner avec les indices de la liste, et renvoie donc souvent les indices de la dimensions -1.

image

commented

Merci @SimonGuillot pour ce retour !
Je pense que ça n'a pas d'impact sur les méthodes comme get_obj_stereotypes, mais que ça impacte en effet get_dimension_stereotypes que l'on utilisait pas guère jusque là.
En tout cas, le dernier imprim'écran est assez parlant.
Le fix du bug est prioritaire @aberanger et ça peut etre bien d'ajouter un cas de test spécifique sur cette fonctionnalité. On ne l'a probablement pas testée :(