GitBucket
docs(lda):added coherence system in comment #57
Merged
leog
merged 13 commits
into
tdm:master
from tdm:lda
on 21 Nov 2023
added coherence metric.
Now, the output of the ws is in "value" and have the best topic with his words and weight
docs(lda):changed output of lda (now in "value") and add "best_topic"
e954696intomasterfromldaon 21 Nov 2023ldabranch on 21 Nov 2023Ce commentaire t'est-il utile ?
À moi, non: il décrit simplement la ligne suivante (appel de la fonction
remove_accents) et reformule avec un synonyme le nom de la fonction.Félicitations: le nom de la fonction est suffisamment bien choisi pour ne plus avoir besoin du commentaire !
M'est avis que c'est l'emplacement idéal pour une docstring, à la place d'un commentaire (mais ce n'est que pure forme).
Pourquoi ne pas utiliser Unidecode pour translittérer les lettres accentuées?
J'ai l'impression que certains caractères sont purement et simplement supprimés.
Si je ne dis pas de bêtise, les caractères sur cette plage "\u0300-\u036f" sont uniquement les caractères "diacritiques combinatoires" : un " ê " s'encode comme " ^e " et seul le " ^ " est supprimé.
Cela évite d'installer une dépendance en plus, mais si tu penses que c'est mieux je peux le modifier en conséquent (re est installé de base avec python 3.8+).
J'avais oublié ce fonctionnement.
Une dépendance de moins, c'est pas mal.
C'est juste au prix d'un code un peu plus long.
Laisse comme ça :)
do not mergelabel on 21 Nov 2023