PetrochukM/PyTorch-NLP

Support loading fasttext model from custom file

Open

#61 ouverte le 3 janv. 2019

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Python (262 forks)batch import
enhancementgood first issuehelp wanted

Métriques du dépôt

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

Description

What if I want to use own pretrained fasttext model (or even commoncrawl model instead of standard wiki one)? E.g. look what they publish now: https://fasttext.cc/docs/en/crawl-vectors.html. Current FastText impl

    url_base = 'https://s3-us-west-1.amazonaws.com/fasttext-vectors/wiki.{}.vec'
    aligned_url_base = 'https://s3.amazonaws.com/arrival/embeddings/wiki.multi.{}.vec'

    def __init__(self, language="en", aligned=False, **kwargs):
        if aligned:
            url = self.aligned_url_base.format(language)
        else:
url = self.url_base.format(language)

doesn't allow you to do such basic stuff.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la classe FastText (probablement dans 'pytorchnlp/fasttext.py'). Ajoutez un paramètre à init pour un chemin de fichier personnalisé. Mettez à jour la logique d'URL pour utiliser le chemin fourni s'il est donné, sinon revenez aux valeurs par défaut. Assurez la compatibilité ascendante. Consultez les commentaires de l'issue pour plus de contexte.
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.
60