pymc-devs/pymc

Unify `distributions/transform.py` and `logprob/transform.py`

Open

#6.908 aberto em 13 de set. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (1.902 forks)batch import
help wantedlogprobmaintenance

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

These modules and, more importantly, respective tests are quite redundant. This is due to part of the functionality being moved to aeppl and then back to pymc via the logprob submodule in the past.

We should probably get rid of distributions/transforms, and possibly split the latter in two files: one for ValueTransforms and RVTransform classes and subclasses, and the other for find_measurable_transform rewrites and related functionality, as that module is getting pretty large.

Guia do colaborador

Direção de pesquisa
Examine os conteúdos de distributions/transform.py e logprob/transform.py para entender a sobreposição. Revise os arquivos de teste correspondentes para identificar redundância. Planeje consolidar os módulos, possivelmente dividindo em dois arquivos: um para as classes ValueTransforms e RVTransform, e outro para as reescritas de find measurable transform.
Pilha de tecnologia
python
Domain
developer experience
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.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 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.
Claro
Prerequisites
Python experienceUnderstanding of PyMCFamiliarity with codebase structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35