Provide configuration descriptions to MTD local reconstruction modules - #51273
Conversation
|
test parameters:
|
|
type ngt |
|
cms-bot internal usage |
|
@cmsbuild, please test |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51273/49872 |
|
A new Pull Request was created by @mmusich for master. It involves the following packages:
@Moanwar, @jfernan2, @mandrenguyen, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
+1 Size: This PR adds an extra 28KB to repository Comparison SummarySummary:
|
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @ftenchini, @mandrenguyen, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
…tion Provide configuration descriptions to MTD local reconstruction modules
…tion Provide configuration descriptions to MTD local reconstruction modules
PR description:
The goal of this PR is to update
MTDUncalibratedRecHitProducerandMTDRecHitProducerin order to make them fully suitable for the HLT usage, by adding afillDescriptionsmethod.As these module make use of plugin factories, additional changes following the line of these instructions were added.
The missing configuration validation was spotted thanks to this script run on
CMSSW_20_1_X_2026-06-21-0000.PR validation:
I've run successfully:
runTheMatrix.py -l 34434.7522 -t 4 -j 8 -i all --ibeosinCMSSW_20_1_X_2026-06-21-0000. (as introduced at Implement MIP Timing Detector reconstruction sequence for Phase-2 HLT #51170)testConfDBParsing_Phase2.pydoes not flag any other MTD modules. The others are long-standing and already reported at Modules not auto-generating a cfi file in cfi-python currently present in the HLT menu (Run 3 and Phase 2) #47275.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:
Not a backport.