pymc-devs/pymc

CheckParameterValues not replaced in inner graphs

Open

#6.697 geöffnet am 28. Apr. 2023

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (1.902 Forks)batch import
help wantedlogprob

Repository-Metriken

Stars
 (7.926 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11T 6h) (12 gemergte PRs in 30 T)

Beschreibung

We include these rewrites when compiling logp graphs, but this won't replace inner logp graphs (e.g, inside Scan or not-inlined OpFromGraph)

https://github.com/pymc-devs/pymc/blob/371472dcf1c44f99a9ad4e7317606404b2a47a0e/pymc/pytensorf.py#L1142-L1144

The example in #6696 suffers from this. Sampling is quite faster when the rewrite is manually added

Contributor Guide

Research-Richtung
Das Problem ist, dass die CheckParameterValues Umschreibung nicht auf innere Graphen angewendet wird (z. B. innerhalb von Scan oder nicht inline OpFromGraph). Der relevante Code befindet sich in pymc/pytensorf.py, Zeilen 1142-1144. Die Lösung würde sicherstellen, dass diese Umschreibung rekursiv auf alle Teilgraphen angewendet wird. Beginnen Sie damit, zu untersuchen, wie Umschreibungen im Kompilierungsprozess angewendet werden, und sehen Sie sich das Beispiel in Issue #6696 an, um den Leistungsunterschied zu reproduzieren. Erwägen Sie, die Anwendungslogik der Umschreibung so zu ändern, dass innere Graphen durchlaufen werden.
Tech Stack
python
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythonbasic PyMC knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55