You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The RooArgSet has a constructor RooArgSet(double) which is accidentally taken by PyROOT here, even though the call to Minos(bool) would be correct.
This caused some bad experience for users who tried to enable Minos for all parameters with Minos(1), resulting in nothing happening (Minos was only attempted for this constant "1" parameter, which of course does nothing).
Not a high priority in this case because the workaround is easy: Minos(True). But it's very possible that users are also bitten in other cases by this faulty overload resolution, so we need to fix it.
The text was updated successfully, but these errors were encountered:
Reproducer:
Should be like in C++:
The function has overloads like this:
The RooArgSet has a constructor
RooArgSet(double)
which is accidentally taken by PyROOT here, even though the call toMinos(bool)
would be correct.This caused some bad experience for users who tried to enable Minos for all parameters with
Minos(1)
, resulting in nothing happening (Minos was only attempted for this constant "1" parameter, which of course does nothing).Not a high priority in this case because the workaround is easy:
Minos(True)
. But it's very possible that users are also bitten in other cases by this faulty overload resolution, so we need to fix it.The text was updated successfully, but these errors were encountered: