facebookresearch/ParlAI

Unify names of some options

Open

#2.170 geöffnet am 18. Nov. 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (2.089 Forks)batch import
Code QualityH1 2020Help WantedMinordonotreap

Repository-Metriken

Stars
 (10.371 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Now that we have upgrade_opt we can do some resolution of the embedding_size and embeddingsize madness we have in seq2seq vs transformer vs memnn etc.

Contributor Guide

Research-Richtung
Durchsuchen Sie die Codebasis nach allen Vorkommen von 'embedding size' und 'embeddingsize'. Diese kommen in seq2seq , transformer und memnn Agenten vor. Verwenden Sie den vorhandenen 'upgrade opt' Mechanismus (wahrscheinlich in options.py oder ähnlich), um die alten inkonsistenten Namen auf einen einzigen einheitlichen Namen zu aliassen. Stellen Sie sicher, dass alle vorhandenen Konfigurationsdateien weiterhin funktionieren, indem Sie die alten Namen abbilden. Konsultieren Sie die Issue Kommentare für weitere Anleitungen zum bevorzugten einheitlichen Namen.
Tech Stack
pythonpytorch
Domain
developer experience
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
PythonParlAI codebase familiarityunderstanding of option naming conventions
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65