First do bounding per LR-system, then calculate percentiles.#15
Conversation
lrmodule/mcmc.py
Outdated
| parameters_h1: dict[str, dict[str, int]] | None, | ||
| distribution_h2: str, | ||
| parameters_h2: dict[str, dict[str, int]] | None, | ||
| bounder: LLRBounder | None, |
There was a problem hiding this comment.
Bij de TLM heet dit 'postprocessing' en is het een Transformer. Is er een reden om dit hier meer restrictief te maken?
There was a problem hiding this comment.
De transform-stap van het bounden gebeurt nu voor alle LR-systemen in één keer; in plaats van systeem voor systeem (zoals bij het fitten nu wel gebeurt, omdat het daar niet anders kan). Als we hier een Transformer willen hebben, dan moet je de transform-stap ook één voor één gaan doen. En voor elk LR-systeem moet je dan een aparte transformer bewaren, ipv van nu alleen de twee arrays met bounds. Dat laatste heb ik nu vast doorgevoerd, het wordt er niet opvallend langzamer door.
Verder is een LLRBounder een TransformerMixin en geen Transformer; bij het TLM doen ze iets met pipelines lijkt het ook. Daar ben ik verder nog niet echt ingedoken. Willen we dat nog gaan doen, of laten we het voor nu even hierbij?
There was a problem hiding this comment.
Als we het bij een bounder: LLRBounder houden, dan stelde @PimMeulensteen voor om als default een ELUBBounder te gebruiken. Bij een postprocessing: Transformer lijkt een default me niet echt zinnig meer.
|
Doe wel even "squash" voor/bij het mergen |
…by one, instead of all at once.
JoBraNFI
left a comment
JoBraNFI
left a comment
There was a problem hiding this comment.
Code itself looks good to me
4 participants
BSZ-237