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 (822 forks)batch import
help wanted

Métricas do repositório

Stars
 (3.396 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Direção de pesquisa
Esta questão envolve substituir todas as ocorrências de 'common.torchtext test case.TorchtextTestCase' por 'torch.testing. internal.common utils.TestCase' nos arquivos de teste. A alteração é direta e localizada nos arquivos test/*.py e test/data/test *.py. Consulte os PRs de exemplo #822 e #823 para o padrão exato. Nenhuma investigação adicional é necessária além da renomeação das importações e usos da classe.
Pilha de tecnologia
pythonpytorch
Domain
testing
Tipo Issue
Refatoração
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitPythonPyTorch
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85