B2OC: fix Lc mass range, introduce a ThOr compliant monotone lipschitz neural network (shifters PLEASE ignore)
patch generated by https://gitlab.cern.ch/lhcb/Moore/-/jobs/19508995
Merge request reports
Activity
assigned to @abertoli
requested review from @shunan
- Resolved by Shunan Zhang
@shunan what is unclear to me in this MR is what I had to do in:
Hlt/Hlt2Conf/python/Hlt2Conf/lines/b_to_open_charm/b_to_dh.py line 30:
from RecoConf.reconstruction_objects import make_pvs_v2
because if in line 107 I do not feed
filter_MON_MVA
withpvs_v2
I get an error that is basically telling me that I am not using the right object forpvs
(see scren picture below)this is also what Leon is doing in his code
Edited by Alessandro Bertolin
- Resolved by Alessandro Bertolin
@shunan you will see that I am keeping both
make_BdToDmK_DmToPimPimKp_MON
andmake_BdToDmK_DmToPimPimKp
because I think we have to show people how to use the new NN but can not yet remove the oldmay be at some point we will decide which one to keep
if we do not want the duplicated line I can put TMVA on make_BdToDmK_DmToPimPimKp and the new NN on make_BdToDmPi_DmToPimPimKp
how do you feel about that ?
- Resolved by Alessandro Bertolin
- Resolved by Alessandro Bertolin
mentioned in commit 745e793f
mentioned in merge request !1414 (merged)