piskvorky/gensim

KeyedVectors TODOs

Open

#2 166 ouverte le 26 août 2018

Voir sur GitHub
 (14 commentaires) (0 réactions) (0 assignés)Python (4 349 forks)batch import
Hacktoberfestdifficulty easydocumentation

Métriques du dépôt

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

Description

Description

Hi! I'm new here :) I was digging through the Doc2Vec code and came across a bunch of TODOs in KeyedVectors.

I was wondering if TODOs such as https://github.com/RaRe-Technologies/gensim/blob/develop/gensim/models/keyedvectors.py#L1601 in KeyedVectors can be implemented using model.infer() like in https://github.com/RaRe-Technologies/gensim/blob/develop/gensim/models/keyedvectors.py#L1801

If so I'll be happy to submit a PR.

Steps/Code/Corpus to Reproduce

N/A

Expected Results

N/A

Actual Results

N/A

Versions

Darwin-15.6.0-x86_64-i386-64bit Python 3.6.4 (default, Jan 6 2018, 11:49:38) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)] NumPy 1.15.1 SciPy 1.1.0 gensim 3.5.0 FAST_VERSION 0

Guide contributeur

Direction de recherche
Examinez les TODO spécifiques dans KeyedVectors à la ligne 1601 et ailleurs dans gensim/models/keyedvectors.py. Vérifiez s'ils peuvent être implémentés en utilisant model.infer() similaire à la ligne 1801. Étudiez l'implémentation existante et la méthode infer. Effectuez les modifications et soumettez une PR.
Stack technique
python
Domaine
machine learning
Type d'issue
Fonctionnalité
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.
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.
75