cms-l1t-offline / cmssw

CMS Offline Software
cms-sw.github.io/cmssw
Apache License 2.0
17 stars 27 forks source link

Adapt ReEmu configuration to run without csctfDigis #546

Open bortigno opened 7 years ago

bortigno commented 7 years ago

ReEmulFromRaw relies on csctfDigid so it won't work for any 2017 data.

https://github.com/cms-l1t-offline/cmssw/blob/l1t-integration-CMSSW_9_2_0/L1Trigger/Configuration/python/customiseReEmul.py#L138

abrinke1 commented 7 years ago

@rekovic @thomreis Should we add "2016" sequences in customiseReEmul.py, as was done for "2015" before? 2017 will see some changes w.r.t. 2016, e.g. OMTF and CPPF unpacking, and no csctfDigis in data.

bortigno commented 7 years ago

hey @abrinke1 it should be taken care by the eras but at the moment I see that for L1trigger 2016 is the same as 2017 https://github.com/cms-l1t-offline/cmssw/blob/l1t-integration-CMSSW_9_2_0/Configuration/Eras/python/Era_Run2_2017_cff.py

Probably a good idea to update that including your suggestion. What do you think @rekovic ?