pymc-devs/pymc

Replace `assert` checks in `logprob.scan` with real error handling

Open

#7.780 aberto em 9 de mai. de 2025

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (1.902 forks)batch import
enhancementshackathonhelp wantedneeds infopytensor

Métricas do repositório

Stars
 (7.926 stars)
Métricas de merge de PR
 (Mesclagem média 11d 6h) (12 fundiu PRs em 30d)

Description

Description

There are two annoying assert checks that I commonly run up against when writing time series models here and here. Other users are hitting them too, see for example here.

It would be much nicer to have a real error message that explains what has gone wrong and why, or at least gives some information about which parts of the graph are going wrong.

I'm not 100% sure how one ends up hitting these asserts, so I tagged this with needs info. It seems to have to do with having multiple outputs from the scan inner function in a custom dist.

Guia do colaborador

Direção de pesquisa
Investigue as condições que disparam as verificações de assert nas linhas 115-116 de pymc/logprob/scan.py. Substitua cada assert por uma mensagem de erro significativa (por exemplo, usando raise TypeError ou ValueError) que explique o problema e sugira possíveis correções. Teste as alterações executando os testes existentes e considerando casos extremos com múltiplas saídas de scan.
Pilha de tecnologia
python
Domain
backend
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.
Principalmente claro
Prerequisites
Pythonpymc
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65