pytorch/text

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

Open

#824 aberto em 11 de jun. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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

Guia do colaborador

Pilha de tecnologia
python
Domain
testing
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
PythonPyTorch basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85
Direção de pesquisa
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.