pymc-devs/pymc

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

Open

#6908 aperta il 13 set 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (1902 fork)batch import
help wantedlogprobmaintenance

Metriche repository

Star
 (7926 star)
Metriche merge PR
 (Merge medio 11g 6h) (12 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina i contenuti di distributions/transform.py e logprob/transform.py per comprendere le sovrapposizioni. Rivedi i file di test corrispondenti per identificare ridondanze. Pianifica di consolidare i moduli, possibilmente suddividendoli in due file: uno per le classi ValueTransforms e RVTransform, e un altro per le riscritture di find measurable transform.
Tech stack
python
Dominio
developer experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python experienceUnderstanding of PyMCFamiliarity with codebase structure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35