-
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
[12.4.X] Introduce TkAlJetHT
#38261
[12.4.X] Introduce TkAlJetHT
#38261
Conversation
A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_12_4_X. It involves the following packages:
@perrotta, @malbouis, @yuanchao, @pmandrik, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @jfernan2, @qliphy, @rvenditti, @micsucmed, @fabiocos, @francescobrivio, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc7a77/25313/summary.html Comparison SummarySummary:
|
+1 |
@cms-sw/orp-l2 this is essentially fully signed |
@cms-sw/orp-l2 as far as I can tell from the cmssw dashboard, the master PR of this backport (#38254) hasn't caused any troubles, could we consider this for inclusion? |
Pull request #38261 was updated. @perrotta, @malbouis, @yuanchao, @pmandrik, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @jfernan2, @qliphy, @rvenditti, @micsucmed, @fabiocos, @francescobrivio, @davidlange6 can you please check and sign again. |
@cmsbuild, please test |
+alca
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc7a77/25364/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see more details here: Comparison SummarySummary:
|
@cms-sw/orp-l2 it would be nice to have this in for 2300 IB too... We already have another update on the GT for 12_4_X |
+1 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_5_X is complete. This pull request will be automatically merged. |
backport of #38254
PR description:
In PR #27307 the
TkAlMinBias
ALCARECO flavour was added to theJetHT
dataset in theautoAlCa
file, in order to have it produced for the Run-2 Ultra-Legacy reprocessing.Operational experience with it has indicated that the resulting PDs are too large to be able tocomfortably distribute them at the target
T2
s, so that workflow is superseded by adding a new producerTkAlJetHT
tailored to be used on theJetHT
sample, which contains a prescale of 10 (1 event of ten input events will be processed). This should alleviate the output size and the burden on disk and operation teams.This is proposed in this PR. As a new
eventSetupPathsKey
(=TkAlJetHTHLT
) is introduced here a new set ofAlCaRecoTriggerBitsRcd
tags:AlCaRecoHLTpaths_2017_offline_v6
which is a superset of the production tagAlCaRecoHLTpaths_2017_offline_v4
, but will all IOVs substituted by new payloads adding theTkAlJetHT
key (used inauto:run2_data*
)AlCaRecoHLTpaths_2021_offline_v2
which is a superset of the production tagAlCaRecoHLTpaths_2021_offline_v1
, but will all IOVs substituted by new payloads adding theTkAlJetHT
key (used inauto:run3_data*
)PR validation:
Run successfully the Tier-0 testing workflow:
runTheMatrix.py -l 1000.0
if this PR is a backport please specify the original PR and why you need to backport that PR:
verbatim backport of #38254