pyro-ppl/pyro

[FR] Error messages should be more specific

Open

#1.453 aberto em 16 de out. de 2018

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)Python (981 forks)batch import
good first issuehelp wantedwarnings & errors

Métricas do repositório

Stars
 (8.211 stars)
Métricas de merge de PR
 (Mesclagem média 10d 19h) (1 fundiu PR em 30d)

Description

Users at PROBPROG 2018 requested more-specific error messages when models are invalid.

Specific improvements include:

  • #1509 Print sample site information when Distribution inputs are invalid. Currently it requires a debugger to see what line of the model is affected.
  • #1534 On error print shapes of sites in the trace: value shape for params, and value,dist,log_prob shapes for sample sites.
  • On error, print contextual information: the current cond_indep_stack, the portion of stack trace under svi.step(), whether the model or guide is being executed, _DIM_ALLOCATOR sate, tracing state, etc. This could be accomplished via a try: except: raise block to add information to a caught error.
  • others?

Guia do colaborador

Direção de pesquisa
Examine o tratamento de erros existente em `pyro/infer/svi.py` e nos módulos de rastreamento relacionados. Estude as implementações dos PRs #1509 e #1534 para adotar seu padrão. A tarefa restante requer a adição de informações contextuais (por exemplo, `cond indep stack`, estado de rastreamento, estado de ` DIM ALLOCATOR`) em um bloco try except ao redor de `svi.step()`. Revise as variáveis de estado interno e determine quais são úteis para imprimir em caso de erro.
Pilha de tecnologia
pythonpytorch
Domain
developer experience
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
PythonPyTorchPyro internalsError handling
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35