icecube / pisa

Monte Carlo-based data analysis
http://icecube.github.io/pisa/
Apache License 2.0
19 stars 47 forks source link

Correct handling of selectable params #687

Closed philippeller closed 2 years ago

philippeller commented 2 years ago

Before, only the current representation of the selectable params would get overwritten in the distributionmaker instantiation, such that params of the same name would point to the same object. Furthermore, the update function in the pipeline would call update on the params instead of the selector. Also, the update in param selector would ignore the values in the underlying dict