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
@sam-may@edjtscott@mhl0116@fabio-mon please take a look at this. If the 2018 labeled file is used for all three years now please remove the old one an update the taggers accordingly, otherwise we should diversify the function in a better why.
Thank you,
simone
The text was updated successfully, but these errors were encountered:
Hello,
in the two libraries there are 2 functions with the same name, i.e., 'selectMuons' and 'selectElectrons'. However, comparing them, the selections differ for some aspect (see below). My proposal is to merge the two libraries in Leptonselection.cc, changing the name of one pair of functions.
selectMuons:
LeptonSelection.cc : the vertex is manually selected as the one which minimizes deltaZ(mu,vtx). This vertex is used to require a minimum deltaZ(mu,trk) and to determine if the muon is tight
LeptonSelection2018.cc : The 0-th vertex is used to determine if the muon is tight and no request on deltaZ(mu,vtx) are applied.
selectElectrons:
LeptonSelection.cc : the vertex is manually selected as the one which minimizes deltaZ(ele,vtx). This vertex is used to require a minimum deltaZ and deltaXY (mu,trk). The photon veto and the isolation are also explicitly applied.
LeptonSelection2018.cc : the method passMVALooseId() (or medium/tight) is used to select the corresponding electrons. No selections on deltaXY or deltaZ (ele,trk) are applied. The photon veto and the isolation are not required (maybe it is applied in a previous/further part of the sequence?).
At the moment there are two libraries related to lepton selections:
https://github.com/cms-analysis/flashgg/blob/dev_legacy_runII/Taggers/src/LeptonSelection.cc
https://github.com/cms-analysis/flashgg/blob/dev_legacy_runII/Taggers/src/LeptonSelection2018.cc
@sam-may @edjtscott @mhl0116 @fabio-mon please take a look at this. If the 2018 labeled file is used for all three years now please remove the old one an update the taggers accordingly, otherwise we should diversify the function in a better why.
Thank you,
simone
The text was updated successfully, but these errors were encountered: