search

cms-analysis / flashgg

20 stars 159 forks source link

Update the VH Leptonic Tags under the STXS framework for UL version #1250

Closed youyingli closed 3 years ago

youyingli commented 3 years ago

Hi @edjtscott , here is the VH Leptonic Tags under the STXS framework for UL version. Note that the WH Leptonic tag is divided 3 STXS bins with ptV = 0_75, 75_150, >150.

There is no problem in my local test. However, could you please cross check when you pull to your branch? Thanks a lot!

MVA training information https://indico.cern.ch/event/961163/contributions/4070079/attachments/2125730/3578901/HggMeeting_19102020.pdf

MVA Boundary optimization https://indico.cern.ch/event/961183/contributions/4079360/attachments/2129857/3586825/HggMeeting_26102020.pdf

edjtscott commented 3 years ago

hi @youyingli, thanks a lot - I will test this week and let you know 👍

edjtscott commented 3 years ago

If you can add the link to the slides you recently presented that would also be handy

edjtscott commented 3 years ago

@youyingli tested this out, seems to work fine!

youyingli commented 3 years ago

Hi @edjtscott , thanks for the check. I think this PR should be merged to the PR which combines all STXS update from your side. So I think this PR can be closed after you make the PR about all STXS update. How do you think?

edjtscott commented 3 years ago

hi You-Ying, yes I agree, all the STXS updates can come together. We can leave this open until I make the final STXS PR 👍

edjtscott commented 3 years ago

Incorporated into PR 1272, will now close this one