-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[14.0.X] Add GEN-SIM information to resonant di-muon TkAl ALCARECO producers #47016
base: CMSSW_14_0_X
Are you sure you want to change the base?
[14.0.X] Add GEN-SIM information to resonant di-muon TkAl ALCARECO producers #47016
Conversation
A new Pull Request was created by @mmusich for CMSSW_14_0_X. It involves the following packages:
@atpathak, @cmsbuild, @consuegs, @jfernan2, @mandrenguyen, @perrotta can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
cms-bot internal usage |
test parameters:
|
@cmsbuild, please test |
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
Hi @cms-sw/orp-l2 , as we have briefly discussed in #46954 (comment) I think it is OK to have this PR in 140X. But I would ask @cms-sw/alca-l2 and @cms-sw/reconstruction-l2 to please do their review of this PR again (even though it is a verbatim backport of the original ones). Then as soon as it is merged and in a new release, @cms-sw/trk-dpg-l2 , please submit a request for a couple of RelVal samples in order to validate the content that this PR introduces. Thanks! |
+1 |
backport of #45357
backport of #45385
PR description:
From the original description at #45357 (comment):
in addition from #45385 (comment)
PR validation:
Run the following command:
runTheMatrix.py --what upgrade -l 12842.0 -t 4 -j 8 --nEvents=100
to produce an input file, then analyzed withSagittaBiasNtuplizer
(introduced back then at PR #44297) by using this patch:and analyzed with:
cmsRun SagittaBiasNtuplizer_cfg.py myfile=file:../../../12842.0_ZMM_13+2024/TkAlDiMuonAndVertex.root
. Finally the resulting output ntuple has been checked for having appropriate branches filled.runTheMatrix.py -l 1001.3
runs with this addition.If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
Verbatim backport of #45357 and #45385 for 2024 MC production purposes.