pytorch/text

[Help Want] Switch from common.torchtext_test_case.TorchtextTestCase to torch.testing._internal.common_utils.TestCase

Open

#824 geöffnet am 11. Juni 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (822 Forks)batch import
help wanted

Repository-Metriken

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

Beschreibung

TestCase.assertEqual supports torch.tensor. Here are a few files required to change:

  • test/*.py
  • test/data/test_*.py

Link example PRs: https://github.com/pytorch/text/pull/822 and https://github.com/pytorch/text/pull/823

Contributor Guide

Research-Richtung
Dieses Issue beinhaltet das Ersetzen aller Vorkommen von 'common.torchtext test case.TorchtextTestCase' durch 'torch.testing. internal.common utils.TestCase' in den Testdateien. Die Änderung ist unkompliziert und auf die Dateien test/*.py und test/data/test *.py beschränkt. Siehe die Beispiel PRs #822 und #823 für das genaue Muster. Über das Umbenennen der Klassenimporte und verwendungen hinaus ist keine weitere Untersuchung erforderlich.
Tech Stack
pythonpytorch
Domain
testing
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitPythonPyTorch
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85