-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add anti-electron tauID for phase2 #31470
Changes from 22 commits
af27465
18025e9
1c5def0
483badf
be8750d
09eab8d
c88b134
1d09b8e
7e6c3a4
7047f3b
bbcfde9
99de1af
6f9cb82
a444e66
03739f8
f8ba5b3
60799fd
c8733a3
32b6a3e
a1c9770
da529a1
eadcf0f
daed709
2ad8a7b
5363318
9277c0b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
/* \class ElectronCollectionMerger | ||
* | ||
* Producer of merged Electron collection | ||
* | ||
* \author: Michal Bluj, NCBJ, Poland | ||
* | ||
*/ | ||
#include "FWCore/Framework/interface/MakerMacros.h" | ||
#include "CommonTools/UtilAlgos/interface/Merger.h" | ||
#include "DataFormats/EgammaCandidates/interface/GsfElectron.h" | ||
#include "DataFormats/EgammaCandidates/interface/GsfElectronFwd.h" | ||
#include "DataFormats/PatCandidates/interface/Electron.h" | ||
|
||
typedef Merger<reco::GsfElectronCollection> GsfElectronCollectionMerger; | ||
DEFINE_FWK_MODULE(GsfElectronCollectionMerger); | ||
|
||
typedef Merger<pat::ElectronCollection> PATElectronCollectionMerger; | ||
DEFINE_FWK_MODULE(PATElectronCollectionMerger); |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -429,7 +429,41 @@ | |
label = cms.untracked.string("RecoTauTag_antiElectronMVA6%s_%s_WP%s" % (antiElectronDiscrMVA6_2017_version, gbrForestName, WP)) | ||
) | ||
) | ||
|
||
|
||
# MVA6 phase2 | ||
antiElectronDiscrMVA_phase2_categories = { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought that the plan was to phase out use of this *FromPrepDB There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This cfi is kept for testing purposes and we would like to have it complete. Additionally what concerns this particular tauID we are still in process with adding it to phase2 GlobalTags. |
||
'0' : "gbr_NoEleMatch_woGwoGSF_BL", | ||
'2' : "gbr_NoEleMatch_wGwoGSF_BL", | ||
'5' : "gbr_woGwGSF_BL", | ||
'7' : "gbr_wGwGSF_BL", | ||
'8' : "gbr_NoEleMatch_woGwoGSF_FWEC", | ||
'9' : "gbr_NoEleMatch_woGwoGSF_VFWEC", | ||
'10' : "gbr_NoEleMatch_wGwoGSF_FWEC", | ||
'11' : "gbr_NoEleMatch_wGwoGSF_VFWEC", | ||
'13' : "gbr_woGwGSF_FWEC", | ||
'14' : "gbr_woGwGSF_VFWEC", | ||
'15' : "gbr_wGwGSF_FWEC", | ||
'16' : "gbr_wGwGSF_VFWEC" | ||
} | ||
antiElectronDiscrMVA_phase2_WPs = [ "Eff98", "Eff90", "Eff80", "Eff70", "Eff60" ] | ||
antiElectronDiscrMVA_phase2_version = "v1" | ||
for category, gbrForestName in antiElectronDiscrMVA_phase2_categories.items(): | ||
loadRecoTauTagMVAsFromPrepDB.toGet.append( | ||
cms.PSet( | ||
record = cms.string('GBRWrapperRcd'), | ||
tag = cms.string("RecoTauTag_antiElectronPhase2MVA6%s_%s" % (antiElectronDiscrMVA_phase2_version, gbrForestName)), | ||
label = cms.untracked.string("RecoTauTag_antiElectronPhase2MVA6%s_%s" % (antiElectronDiscrMVA_phase2_version, gbrForestName)) | ||
) | ||
) | ||
for WP in antiElectronDiscrMVA_phase2_WPs: | ||
loadRecoTauTagMVAsFromPrepDB.toGet.append( | ||
cms.PSet( | ||
record = cms.string('PhysicsTGraphPayloadRcd'), | ||
tag = cms.string("RecoTauTag_antiElectronPhase2MVA6%s_%s_WP%s" % (antiElectronDiscrMVA_phase2_version, gbrForestName, WP)), | ||
label = cms.untracked.string("RecoTauTag_antiElectronPhase2MVA6%s_%s_WP%s" % (antiElectronDiscrMVA_phase2_version, gbrForestName, WP)) | ||
) | ||
) | ||
|
||
#### | ||
# register anti-muon discriminator MVA | ||
antiMuonDiscrMVA_WPs = [ "eff99_5", "eff99_0", "eff98_0" ] | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be done in
phase2_common.toModify(
?if there is a chance that the list of IDs will grow, it would be better to factor out these calls to a python function, preferably, defined in the same place where all these modules are defined as well. This will decrease unnecessary bloat of the master miniAOD_tools
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usage of condition
if
was selected in similar case with other tauID, please check discussion following this: #30341 (comment).An alternative way would be direct import of definitions for a cff w/o use of runTauID python tool, but not necessarily simpler.