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
Fix of L1EMTF modifier for legacy16 #29287
Fix of L1EMTF modifier for legacy16 #29287
Conversation
A new Pull Request was created by @srimanob (Phat Srimanobhas) for CMSSW_10_6_X. It involves the following packages: Configuration/Eras @benkrikler, @silviodonato, @cmsbuild, @rekovic, @franzoni, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@rekovic @jiafulow @abrinke1 What I concern is not L1Repack on data, but MC will also affect. With following cmsDriver (*) used in official MC campaign, we will get wrong config on simEmtfDigis if new release is used.
(*) |
@srimanob I just replied to your email with the subject "Additional fix to EMTF emulation for UL2016". In short, I believe the stage2L1Trigger_EMTF2016 modifier that you created is indeed what we needed. The previous EMTF PRs were not really correct. Thanks for preparing the fix. Regarding your message above, I did the following: export SCRAM_ARCH=slc7_amd64_gcc700
source /cvmfs/cms.cern.ch/cmsset_default.sh
cmsrel CMSSW_10_6_X_2020-03-24-1100
cd CMSSW_10_6_X_2020-03-24-1100/src
cmsenv
git cms-merge-topic srimanob:10_6_FixL1EMTFModifierForLegacy16
scram b and ran the cmsDriver command that you provided. In the "dump" python cfg file, this is what I see: $ python -i JME-RunIISummer19UL18DIGI-00002_1_cfg.py
>>> process.simEmtfDigis.Era
cms.string('Run2_2018')
>>> process.simEmtfDigis.RPCEnable
cms.bool(True) This is the correct behavior for MC with era=Run2_2018. So I think the PR is good. Is there anything that I'm missing? |
@jiafulow Thanks for confirmation |
merge |
+1 |
thanks @srimanob |
PR description:
This PR is to detach the stage2L1Trigger introduced in #29156 for L1EMTF. "stage2L1Trigger" is used for all 2016, 2017, 2018, so changing it means to change all. However, we would like to avoid the change in 2017, 2018 as Legacy program starts.
This PR is for 10_6 only. For master, we need to detach stage2L1Trigger_2017 also.
PR validation:
Dumping configurations from following workflows and compare with 10_6_10, results are expected:
Dump configs can be found at
/afs/cern.ch/user/s/srimanob/public/L1EMTF/afterFix
2016:
2017, 2018