pytorch/text

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

Ouverte

#824 ouverte le 11 juin 2020

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Python (822 forks)batch import
help wanted

Métriques du dépôt

Stars
 (3 396 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Direction de recherche
Ce problème implique de remplacer toutes les occurrences de 'common.torchtext test case.TorchtextTestCase' par 'torch.testing. internal.common utils.TestCase' dans les fichiers de test. Le changement est simple et localisé aux fichiers test/*.py et test/data/test *.py. Reportez vous aux PR d'exemple #822 et #823 pour le modèle exact. Aucune investigation supplémentaire n'est nécessaire au delà du renommage des imports et des utilisations de la classe.
Stack technique
pythonpytorch
Domaine
testing
Type d'issue
Refactorisation
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.
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
GitPythonPyTorch
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85