search

LHCfitNikhef / smefit_release

SMEFiT a Standard Model Effective Field Theory fitter
GNU General Public License v3.0
6 stars 1 forks source link

Bug in plot_posteriors #46

Closed jacoterh closed 1 year ago

jacoterh commented 1 year ago

Hi Giacomo,

The following checks whether disjointed_lists is None, but this will always fail because of the line here

https://github.com/LHCfitNikhef/smefit_release/blob/775df86c7a6499b70fda5e308b089456d8e03927/src/smefit/analyze/coefficients_utils.py#L340

It is probably a quick fix.

Thanks!

giacomomagni commented 1 year ago

I think I recently fix this on the smefit_uv branch: https://github.com/LHCfitNikhef/smefit_release/blob/4f1b7c988e4f20caebfa59736fdf6b46e196dec7/src/smefit/analyze/coefficients_utils.py#L340

is this correct?

jacoterh commented 1 year ago

Yes that fixes it, I will open a separate branch and link a PR to main!

giacomomagni commented 1 year ago

you can commit directly to main thanks

jacoterh commented 1 year ago

Fixed in defee38