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
EGM IDs for MiniAOD Reprocessing (76X) #10619
Conversation
A new Pull Request was created by @lgray (Lindsey Gray) for CMSSW_7_6_X. EGM IDs for MiniAOD Reprocessing (76X) It involves the following packages: DataFormats/EgammaCandidates @cmsbuild, @cvuosalo, @vadler, @monttj, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
switchOnVIDElectronIdProducer(process,DataFormat.MiniAOD) | ||
process.egmGsfElectronIDs.physicsObjectSrc = \ | ||
cms.InputTag("reducedEgamma","reducedGedGsfElectrons") | ||
process.electronMVAValueMapProducer.src = \ | ||
cms.InputTag('reducedEgamma','reducedGedGsfElectrons') | ||
for idmod in electron_ids: | ||
setupAllVIDIdsInModule(process,idmod,setupVIDElectronSelection) | ||
setupAllVIDIdsInModule(process,idmod,setupVIDElectronSelection,None,False) |
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.
False means not to add to UserData? Where do these values go then? just empty space or are they consumed somewhere (lazy to parse the underlying code)
it may help review if this one is rebased ... running short++ matrix now |
@slava77 ok I'll rebase it in a bit |
…stinguish different cases of the MVA of the same class configured with different weights/tunings.
The tests are being triggered in jenkins. |
Nope DAS is still dead for jenkins it seems. The present test will fail. |
+1
|
-1 Tested at: 6b8b134 ---> test testRecoMETMETProducers had ERRORS you can see the results of the tests here: |
@cmsbuild please test |
The tests are being triggered in jenkins. |
EGM IDs for MiniAOD Reprocessing (76X)
Forward port of #10598.
Depends on #10618.
Weight files are absent from this PR.