MTD geometry: add BTL V3 (one crystal thickness only) to MTD scenario I17 (global scenario D105)

[fabiocos]

PR description:

This PR adds a new BTL scenario V3 on top of the just integrated MTD scenario I17, added in #43407 . This new configuration, developed by @pauris123 , differs from V2 for the crystal thickness, set everywhere to 3.75 mm, as it will be in reality (purchase order ongoing). This is the opportunity to drop the crystal thickness from the geometry construction, and have 6 equal readout units along z.

The numbering scheme for the time being is just adapted to the existing BTLDetId, creating an effective crystal thickness index, so as to postpone the major update to the moment in which the BTL V1 can be finally dropped, allowing us to optimize the bit fields structure for a final configuration.

PR validation:

All basic geometry checks (when geometry scenario is adapted) look reasonable, and a complete workflow runs. No overlap is detected at ideal geometry level, and visual inspection in fireworks appears ok.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43553/38154

• This PR adds an extra 56KB to repository

• There are other open Pull requests which might conflict with changes you have proposed:

  • File Configuration/Geometry/python/dict2026Geometry.py modified in PR(s): revert PR 40443 w/o introducing bricked pixels #42454

[cmsbuild]

A new Pull Request was created by @fabiocos (Fabio Cossutti) for master.

It involves the following packages:

• Configuration/Geometry (upgrade, geometry)
• DataFormats/ForwardDetId (upgrade, simulation)
• Geometry/CMSCommonData (upgrade, geometry)
• Geometry/MTDCommonData (upgrade, geometry)
• Geometry/MTDGeometryBuilder (upgrade, geometry)
• Geometry/MTDNumberingBuilder (upgrade, geometry)
• Geometry/MTDSimData (upgrade, geometry)
• RecoMTD/DetLayers (upgrade, reconstruction)

@Dr15Jones, @jfernan2, @mandrenguyen, @srimanob, @cmsbuild, @bsunanda, @subirsarkar, @makortel, @mdhildreth, @civanch can you please review it and eventually sign? Thanks.
@vargasa, @Martin-Grunewald, @missirol, @bsunanda, @slomeo, @rovere, @youyingli, @apsallid this is something you requested to watch as well.
@sextonkennedy, @rappoccio, @antoniovilela you are the release manager for this.

cms-bot commands are listed here

[fabiocos]

please test workflow 27634.0

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c99e72/36440/summary.html
COMMIT: 71bd324
CMSSW: CMSSW_14_0_X_2023-12-12-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/43553/36440/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @24LopezR ETL Geometry V8 #43407
• @bsunanda Run3-sim152Z Take care of difference between DDD and DD4hep in SteppingAction classes (replace the PR #43499) #43546
• @pallabidas emulator for Phase-2 GCT calo jets and taus #43245
• @Dr15Jones Added transform info to Tracer Log #43528
• @mmusich Add BeamSpotOnlineShifter class and related configuration #43505
• @stahlleiton Run3 era for 2023 PbPb UPC re-reco #43378

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c99e72/36440/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c99e72/36440/git-merge-result

Comparison Summary

Summary:

• You potentially removed 88 lines from the logs
• Reco comparison results: 24 differences found in the comparisons
• DQMHistoTests: Total files compared: 50
• DQMHistoTests: Total histograms compared: 3429858
• DQMHistoTests: Total failures: 1214
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3428622
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 49 files compared)
• Checked 214 log files, 167 edm output root files, 50 DQM output files
• TriggerResults: no differences found

[srimanob]

+Upgrade

[jfernan2]

+1

[civanch]

+1

[cmsbuild]

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. @rappoccio, @sextonkennedy, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2)

[rappoccio]

+1