pytorch/text

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

Open

#824 ouverte le 11 juin 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Python (3 396 stars) (822 forks)batch import
help wanted

Description

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

Guide contributeur

Stack technique
python
Domaine
testing
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
PythonPyTorch basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
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.