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 (3.396 Stars) (822 Forks)batch import
help wanted

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

Tech Stack
python
Domain
testing
Issue Type
refactor
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
PythonPyTorch basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
This issue involves replacing all occurrences of 'common.torchtext test case.TorchtextTestCase' with 'torch.testing. internal.common utils.TestCase' in the test files. The change is straightforward and localized to test/*.py and test/data/test *.py. Refer to example PRs #822 and #823 for the exact pattern. No additional investigation is needed beyond renaming the class imports and usages.