-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Save L1 stage2 info in offline data tiers (81X) #14051
Changes from 2 commits
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 |
---|---|---|
@@ -1,4 +1,5 @@ | ||
import FWCore.ParameterSet.Config as cms | ||
from Configuration.StandardSequences.Eras import eras | ||
|
||
# RAW content | ||
L1TriggerRAW = cms.PSet( | ||
|
@@ -93,3 +94,20 @@ | |
'keep LumiSummary_lumiProducer_*_*') | ||
) | ||
|
||
|
||
if eras.stage2L1Trigger.isChosen(): | ||
# stage 2 L1 trigger | ||
l1Stage2Digis = [ | ||
'keep *_gtStage2Digis__*', | ||
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. there should be a star between the two underscores.... 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 did not put the star on purpose, as my understanding from the email thread was that for the GT we wanted only the bits ( 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. Are you sure it is the correct syntax in this case? |
||
'keep *_gmtStage2Digis_Muon_*', | ||
'keep *_caloStage2Digis_Jet_*', | ||
'keep *_caloStage2Digis_Tau_*', | ||
'keep *_caloStage2Digis_EGamma_*', | ||
'keep *_caloStage2Digis_EtSum_*', | ||
] | ||
# adding them to all places where we had l1extraParticles | ||
L1TriggerRAWDEBUG.outputCommands += l1Stage2Digis | ||
L1TriggerRECO.outputCommands += l1Stage2Digis | ||
L1TriggerAOD.outputCommands += l1Stage2Digis | ||
L1TriggerFEVTDEBUG.outputCommands += l1Stage2Digis | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -47,8 +47,17 @@ | |
'keep patPackedTriggerPrescales_patTrigger__*', | ||
'keep patPackedTriggerPrescales_patTrigger_l1max_*', | ||
'keep patPackedTriggerPrescales_patTrigger_l1min_*', | ||
# old L1 trigger | ||
'keep *_l1extraParticles_*_*', | ||
'keep L1GlobalTriggerReadoutRecord_gtDigis_*_*', | ||
# stage 2 L1 trigger | ||
'keep *_gtStage2Digis__*', | ||
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. there should be a star between the two underscores.... |
||
'keep *_gmtStage2Digis_Muon_*', | ||
'keep *_caloStage2Digis_Jet_*', | ||
'keep *_caloStage2Digis_Tau_*', | ||
'keep *_caloStage2Digis_EGamma_*', | ||
'keep *_caloStage2Digis_EtSum_*', | ||
# HLT | ||
'keep *_TriggerResults_*_HLT', | ||
'keep *_TriggerResults_*_*', # for MET filters (a catch all for the moment, but ideally it should be only the current process) | ||
'keep patPackedCandidates_lostTracks_*_*', | ||
|
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.
this syntax is strongly discouraged
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.
I took it from another L1Trigger cff file, assuming it was the right thing to do.
What's the right syntax?
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.
Sorry, I missed your question earlier.
use simply toModify with a function